[PATCH] D29299: [X86][Tablegen] Fixing register defs and uses for the ENTER instruction.
Chris Ray via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 30 17:51:09 PST 2017
cray added inline comments.
================
Comment at: lib/Target/X86/X86InstrInfo.td:1080
// Constructing a stack frame.
-def ENTER : Ii16<0xC8, RawFrmImm8, (outs), (ins i16imm:$len, i8imm:$lvl),
- "enter\t$len, $lvl", [], IIC_ENTER>, Sched<[WriteMicrocoded]>;
+let Defs = [EBP, ESP], Uses = [EBP, ESP], mayLoad = 1, hasSideEffects=0 in
+def ENTER : Ii16<0xC8, RawFrmImm8, (outs), (ins i16imm:$len, i8imm:$lvl),
----------------
craig.topper wrote:
> Why mayLoad? Shouldn't it be mayStore?
Nice catch! I will fix this.
Repository:
rL LLVM
https://reviews.llvm.org/D29299
More information about the llvm-commits
mailing list