craig.topper added inline comments.
================
Comment at: llvm/lib/Target/X86/AsmParser/X86Operand.h:386
return false;
+ if (getMemDisp()->getKind() == llvm::MCExpr::SymbolRef)
+ return true;
----------------
Wouldn't this allow "symbol + %eax" to pick the 64-bit instruction which is
what the original change was trying to avoid?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D151863/new/
https://reviews.llvm.org/D151863
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
