[PATCH] D60228: [X86] Merge the different Jcc instructions for each condition code into single instructions that store the condition code as an operand.
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 4 08:21:01 PDT 2019
RKSimon added inline comments.
================
Comment at: llvm/lib/Target/X86/MCTargetDesc/X86MCCodeEmitter.cpp:1331
+ EmitByte(BaseOpcode, CurByte, OS);
+ }
----------------
craig.topper wrote:
> RKSimon wrote:
> > Worth doing a LLVM_FALLTHROUGH here?
> By hoisting Adder out of the switch and initializing to 0?
Yeah - "OpcodeOffset" or something.......
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60228/new/
https://reviews.llvm.org/D60228
More information about the llvm-commits
mailing list