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

Unified Diff: src/trusted/validator_ragel/decoder_internal.h

Issue 11000033: Move validator_x86_XX.rl out of unreviewed. (Closed) Base URL: svn://svn.chromium.org/native_client/trunk/src/native_client/
Patch Set: Created 7 years, 9 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
« no previous file with comments | « src/trusted/validator_ragel/decoder.h ('k') | src/trusted/validator_ragel/decoder_x86_32.rl » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/trusted/validator_ragel/decoder_internal.h
===================================================================
--- src/trusted/validator_ragel/decoder_internal.h (revision 11020)
+++ src/trusted/validator_ragel/decoder_internal.h (working copy)
@@ -12,41 +12,45 @@
#ifndef NATIVE_CLIENT_SRC_TRUSTED_VALIDATOR_RAGEL_DECODER_INTERNAL_H_
#define NATIVE_CLIENT_SRC_TRUSTED_VALIDATOR_RAGEL_DECODER_INTERNAL_H_
-#include "native_client/src/trusted/validator_ragel/unreviewed/decoding.h"
+#include "native_client/src/trusted/validator_ragel/decoding.h"
/*
* Set of macroses used in actions defined in parse_instruction.rl to pull
* parts of the instruction from a byte stream and store them for future use.
*/
-#define GET_REX_PREFIX() instruction.prefix.rex
-#define GET_VEX_PREFIX2() vex_prefix2
#define GET_VEX_PREFIX3() vex_prefix3
-#define SET_VEX_PREFIX3(P) vex_prefix3 = (P)
-#define SET_DATA16_PREFIX(S) instruction.prefix.data16 = (S)
-#define SET_LOCK_PREFIX(S) instruction.prefix.lock = (S)
-#define SET_REPZ_PREFIX(S) instruction.prefix.repz = (S)
-#define SET_REPNZ_PREFIX(S) instruction.prefix.repnz = (S)
-#define SET_BRANCH_TAKEN(S) instruction.prefix.branch_taken = (S)
-#define SET_BRANCH_NOT_TAKEN(S) instruction.prefix.branch_not_taken = (S)
-#define SET_INSTRUCTION_NAME(N) instruction.name = (N)
-#define GET_OPERAND_NAME(N) instruction.operands[(N)].name
-#define SET_OPERAND_NAME(N, S) instruction.operands[(N)].name = (S)
-#define SET_OPERAND_FORMAT(N, S) instruction.operands[(N)].format = (S)
-#define SET_OPERANDS_COUNT(N) instruction.operands_count = (N)
-#define SET_MODRM_BASE(N) instruction.rm.base = (N)
-#define SET_MODRM_INDEX(N) instruction.rm.index = (N)
-#define SET_MODRM_SCALE(S) instruction.rm.scale = (S)
-#define SET_DISP_TYPE(T) instruction.rm.disp_type = (T)
-#define SET_DISP_PTR(P) \
- instruction.rm.offset = DecodeDisplacementValue(instruction.rm.disp_type, (P))
-#define SET_IMM_TYPE(T) imm_operand = (T)
-#define SET_IMM_PTR(P) \
- instruction.imm[0] = DecodeImmediateValue(imm_operand, (P))
-#define SET_IMM2_TYPE(T) imm2_operand = (T)
-#define SET_IMM2_PTR(P) \
- instruction.imm[1] = DecodeImmediateValue(imm2_operand, (P))
-#define SET_CPU_FEATURE(F)
-#define SET_ATT_INSTRUCTION_SUFFIX(S) instruction.att_instruction_suffix = (S)
+#define SET_VEX_PREFIX3(PREFIX_BYTE) vex_prefix3 = (PREFIX_BYTE)
+#define SET_DATA16_PREFIX(STATUS) instruction.prefix.data16 = (STATUS)
+#define SET_LOCK_PREFIX(STATUS) instruction.prefix.lock = (STATUS)
+#define SET_REPZ_PREFIX(STATUS) instruction.prefix.repz = (STATUS)
+#define SET_REPNZ_PREFIX(STATUS) instruction.prefix.repnz = (STATUS)
+#define SET_BRANCH_TAKEN(STATUS) instruction.prefix.branch_taken = (STATUS)
+#define SET_BRANCH_NOT_TAKEN(STATUS) \
+ instruction.prefix.branch_not_taken = (STATUS)
+#define SET_INSTRUCTION_NAME(NAME) instruction.name = (NAME)
+#define GET_OPERAND_NAME(INDEX) instruction.operands[(INDEX)].name
+#define SET_OPERAND_NAME(INDEX, NAME) \
+ instruction.operands[(INDEX)].name = (NAME)
+#define SET_OPERAND_FORMAT(INDEX, FORMAT) \
+ instruction.operands[(INDEX)].format = (FORMAT)
+#define SET_OPERANDS_COUNT(COUNT) instruction.operands_count = (COUNT)
+#define SET_MODRM_BASE(REG_NUMBER) instruction.rm.base = (REG_NUMBER)
+#define SET_MODRM_INDEX(REG_NUMBER) instruction.rm.index = (REG_NUMBER)
+#define SET_MODRM_SCALE(VALUE) instruction.rm.scale = (VALUE)
+#define SET_DISPLACEMENT_FORMAT(FORMAT) instruction.rm.disp_type = (FORMAT)
+#define SET_DISPLACEMENT_POINTER(POINTER) \
+ instruction.rm.offset = \
+ DecodeDisplacementValue(instruction.rm.disp_type, (POINTER))
+#define SET_IMMEDIATE_FORMAT(FORMAT) imm_operand = (FORMAT)
+#define SET_IMMEDIATE_POINTER(POINTER) \
+ instruction.imm[0] = DecodeImmediateValue(imm_operand, (POINTER))
+#define SET_SECOND_IMMEDIATE_FORMAT(FORMAT) imm2_operand = (FORMAT)
+#define SET_SECOND_IMMEDIATE_POINTER(POINTER) \
+ instruction.imm[1] = DecodeImmediateValue(imm2_operand, (POINTER))
+/* We don't support CPUID feature detection in decoder. */
+#define SET_CPU_FEATURE(FEATURE)
+#define SET_ATT_INSTRUCTION_SUFFIX(SUFFIX_STRING) \
+ instruction.att_instruction_suffix = (SUFFIX_STRING)
/*
* Immediate mode: size of the instruction's immediate operand. Note that there
« no previous file with comments | « src/trusted/validator_ragel/decoder.h ('k') | src/trusted/validator_ragel/decoder_x86_32.rl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698