Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(269)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: src/trusted/validator_ragel/unreviewed/decoder-test.c

Issue 9968039: Add ragel machine generators to SCONS (Closed) Base URL: svn://svn.chromium.org/native_client/trunk/src/native_client/
Patch Set: Created 8 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« src/trusted/validator_ragel/unreviewed/decoder.h ('K') | « src/trusted/validator_ragel/unreviewed/decoder.h ('k') | src/trusted/validator_ragel/unreviewed/decoder-x86_32.rl » ('j') | src/trusted/validator_ragel/unreviewed/decoder-x86_32.rl » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/trusted/validator_ragel/unreviewed/decoder-test.c
===================================================================
--- src/trusted/validator_ragel/unreviewed/decoder-test.c (revision 8170)
+++ src/trusted/validator_ragel/unreviewed/decoder-test.c (working copy)
@@ -5,26 +5,33 @@
*/
#include <assert.h>
-#include <elf.h>
-#include <inttypes.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
-#include "decoder.h"
+#include "native_client/src/include/elf32.h"
+#include "native_client/src/include/elf64.h"
+#include "native_client/src/trusted/validator_ragel/unreviewed/decoder.h"
+
#undef TRUE
#define TRUE 1
#undef FALSE
#define FALSE 0
+#ifdef __GNUC__
+/* We only use argyuments in assert(3) and that generates warnings in release
pasko-google - do not use 2012/04/05 14:16:38 arguments As far as I can see the convention in T
khim 2012/04/05 16:38:43 Done.
+ mode. */
+#pragma GCC diagnostic ignored "-Wunused-parameter"
+#endif
+
static void CheckBounds(unsigned char *data, size_t data_size,
void *ptr, size_t inside_size) {
assert(data <= (unsigned char *) ptr);
assert((unsigned char *) ptr + inside_size <= data + data_size);
}
-void ReadFile(const char *filename, uint8_t **result, size_t *result_size) {
+void ReadImage(const char *filename, uint8_t **result, size_t *result_size) {
FILE *fp;
uint8_t *data;
size_t file_size;
@@ -76,7 +83,7 @@
int show_name_suffix = FALSE;
int empty_rex_prefix_ok = FALSE;
#define print_name(x) (printf((x)), shown_name += strlen((x)))
- int shown_name = 0;
+ size_t shown_name = 0;
int i, operand_type;
/* "fwait" is nasty: any number of them will be included in other X87
@@ -530,7 +537,7 @@
if ((instruction->rm.base >= REG_R8) &&
(instruction->rm.base <= REG_R15)) {
++rex_bits;
- } else if ((instruction->rm.base == REG_NONE) ||
+ } else if ((instruction->rm.base == NO_REG) ||
(instruction->rm.base == REG_RIP)) {
++maybe_rex_bits;
}
@@ -776,11 +783,12 @@
}
}
{
+ size_t i;
/* Print branch hint suffixes for conditional jump instructions (Jcc). */
const char* jcc_jumps[] = {
"ja", "jae", "jbe", "jb", "je", "jg", "jge", "jle",
"jl", "jne", "jno", "jnp", "jns", "jo", "jp", "js", NULL};
- for (int i = 0; jcc_jumps[i] != NULL; ++i) {
+ for (i = 0; jcc_jumps[i] != NULL; ++i) {
if (!strcmp(instruction_name, jcc_jumps[i])) {
if (instruction->prefix.branch_not_taken) {
print_name(",pn");
@@ -1149,11 +1157,11 @@
break;
case REG_RM: {
if (instruction->rm.offset) {
- printf("0x%"PRIx64, instruction->rm.offset);
+ printf("0x%"NACL_PRIx64, instruction->rm.offset);
}
if (((struct DecodeState *)userdata)->ia32_mode) {
- if ((instruction->rm.base != REG_NONE) ||
- (instruction->rm.index != REG_NONE) ||
+ if ((instruction->rm.base != NO_REG) ||
+ (instruction->rm.index != NO_REG) ||
(instruction->rm.scale != 0)) {
printf("(");
}
@@ -1166,7 +1174,7 @@
case REG_RBP: printf("%%ebp"); break;
case REG_RSI: printf("%%esi"); break;
case REG_RDI: printf("%%edi"); break;
- case REG_NONE: break;
+ case NO_REG: break;
case REG_R8:
case REG_R9:
case REG_R10:
@@ -1201,7 +1209,7 @@
(instruction->rm.scale != 0))
printf(",%%eiz,%d",1<<instruction->rm.scale);
break;
- case REG_NONE: break;
+ case NO_REG: break;
case REG_R8:
case REG_R9:
case REG_R10:
@@ -1222,13 +1230,13 @@
case JMP_TO:
assert(FALSE);
}
- if ((instruction->rm.base != REG_NONE) ||
- (instruction->rm.index != REG_NONE) ||
+ if ((instruction->rm.base != NO_REG) ||
+ (instruction->rm.index != NO_REG) ||
(instruction->rm.scale != 0)) {
printf(")");
}
} else {
- if ((instruction->rm.base != REG_NONE) ||
+ if ((instruction->rm.base != NO_REG) ||
(instruction->rm.index != REG_RIZ) ||
(instruction->rm.scale != 0)) {
printf("(");
@@ -1251,7 +1259,7 @@
case REG_R14: printf("%%r14"); break;
case REG_R15: printf("%%r15"); break;
case REG_RIP: printf("%%rip"); print_rip = TRUE; break;
- case REG_NONE: break;
+ case NO_REG: break;
case REG_RM:
case REG_RIZ:
case REG_IMM:
@@ -1281,13 +1289,13 @@
case REG_R13: printf(",%%r13,%d",1<<instruction->rm.scale); break;
case REG_R14: printf(",%%r14,%d",1<<instruction->rm.scale); break;
case REG_R15: printf(",%%r15,%d",1<<instruction->rm.scale); break;
- case REG_RIZ: if (((instruction->rm.base != REG_NONE) &&
+ case REG_RIZ: if (((instruction->rm.base != NO_REG) &&
(instruction->rm.base != REG_RSP) &&
(instruction->rm.base != REG_R12)) ||
(instruction->rm.scale != 0))
printf(",%%riz,%d",1<<instruction->rm.scale);
break;
- case REG_NONE: break;
+ case NO_REG: break;
case REG_RM:
case REG_RIP:
case REG_IMM:
@@ -1300,7 +1308,7 @@
case JMP_TO:
assert(FALSE);
}
- if ((instruction->rm.base != REG_NONE) ||
+ if ((instruction->rm.base != NO_REG) ||
(instruction->rm.index != REG_RIZ) ||
(instruction->rm.scale != 0)) {
printf(")");
@@ -1309,11 +1317,11 @@
}
break;
case REG_IMM: {
- printf("$0x%"PRIx64,instruction->imm[0]);
+ printf("$0x%"NACL_PRIx64,instruction->imm[0]);
break;
}
case REG_IMM2: {
- printf("$0x%"PRIx64,instruction->imm[1]);
+ printf("$0x%"NACL_PRIx64,instruction->imm[1]);
break;
}
case REG_PORT_DX: printf("(%%dx)"); break;
@@ -1336,28 +1344,28 @@
}
break;
case JMP_TO: if (instruction->operands[0].type == OperandSize16bit)
- printf("0x%zx", ((end + instruction->rm.offset -
+ printf("0x%"NACL_PRIxS, ((end + instruction->rm.offset -
(((struct DecodeState *)userdata)->offset)) & 0xffff));
else
- printf("0x%zx", (end + instruction->rm.offset -
+ printf("0x%"NACL_PRIxS, (end + instruction->rm.offset -
(((struct DecodeState *)userdata)->offset)));
break;
case REG_RIP:
case REG_RIZ:
- case REG_NONE:
+ case NO_REG:
assert(FALSE);
}
delimeter = ',';
}
if (print_rip) {
- printf(" # 0x%8"PRIx64,
+ printf(" # 0x%8"NACL_PRIx64,
(uint64_t) (end + instruction->rm.offset -
(((struct DecodeState *)userdata)->offset)));
}
printf("\n");
begin += 7;
while (begin < end) {
- printf("%*"PRIx64":\t", ((struct DecodeState *)userdata)->width,
+ printf("%*"NACL_PRIx64":\t", ((struct DecodeState *)userdata)->width,
(uint64_t) (begin - (((struct DecodeState *)userdata)->offset)));
for (p = begin; p < begin + 7; ++p) {
if (p >= end) {
@@ -1376,7 +1384,7 @@
}
void ProcessError (const uint8_t *ptr, void *userdata) {
- printf("rejected at %"PRIx64" (byte 0x%02"PRIx32")\n",
+ printf("rejected at %"NACL_PRIx64" (byte 0x%02"NACL_PRIx32")\n",
(uint64_t) (ptr - (((struct DecodeState *)userdata)->offset)),
*ptr);
}
@@ -1386,7 +1394,7 @@
uint8_t *data;
int count;
- ReadFile(filename, &data, &data_size);
+ ReadImage(filename, &data, &data_size);
if (data[4] == 1) {
for (count = 0; count < repeat_count; ++count) {
Elf32_Ehdr *header;
« src/trusted/validator_ragel/unreviewed/decoder.h ('K') | « src/trusted/validator_ragel/unreviewed/decoder.h ('k') | src/trusted/validator_ragel/unreviewed/decoder-x86_32.rl » ('j') | src/trusted/validator_ragel/unreviewed/decoder-x86_32.rl » ('J')

Powered by Google App Engine
This is Rietveld 408576698