APFv7: remove spurious assert in LD{B,H,W}{,X}
This is already checked by following 2 asserts a few lines lower.
Before:
text data bss dec hex filename
4364 0 0 4364 110c apf_interpreter.arm.o
text data bss dec hex filename
5623 0 0 5623 15f7 apf_interpreter.x86.o
After:
text data bss dec hex filename
4356 0 0 4356 1104 apf_interpreter.arm.o
text data bss dec hex filename
5606 0 0 5606 15e6 apf_interpreter.x86.o
Generate via:
cd //hardware/google/apf/devtools
./mk
vi ../v7/apf_interpreter_source.c
../v7/apf_interpreter_assemble.sh
./mk
Test: manual
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: I3cdf8908a63788ef8d7a33920f137e745bbf2ee0
diff --git a/v7/apf_interpreter.c b/v7/apf_interpreter.c
index 635ee3e..c014ca4 100644
--- a/v7/apf_interpreter.c
+++ b/v7/apf_interpreter.c
@@ -766,7 +766,6 @@
u32 offs = imm;
/* Note: this can overflow and actually decrease offs. */
if (opcode >= LDBX_OPCODE) offs += ctx->R[1];
- ASSERT_IN_PACKET_BOUNDS(offs);
switch (opcode) {
case LDB_OPCODE:
case LDBX_OPCODE:
diff --git a/v7/apf_interpreter_source.c b/v7/apf_interpreter_source.c
index 6a70472..2596887 100644
--- a/v7/apf_interpreter_source.c
+++ b/v7/apf_interpreter_source.c
@@ -220,7 +220,6 @@
u32 offs = imm;
// Note: this can overflow and actually decrease offs.
if (opcode >= LDBX_OPCODE) offs += ctx->R[1];
- ASSERT_IN_PACKET_BOUNDS(offs);
switch (opcode) {
case LDB_OPCODE:
case LDBX_OPCODE:
