[PATCH] D28689: X86: Produce @ABS8 symbol modifiers for absolute symbols in range [0, 256).

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 31 10:49:40 PST 2017


RKSimon added inline comments.


================
Comment at: llvm/lib/Target/X86/X86Subtarget.cpp:100
+      // See if we can use the 8-bit immediate form.
+      if (CR->getUnsignedMax().ult(256))
+        return X86II::MO_ABS8;
----------------
Should we treating this as an unsigned value? D28688 suggested its sign-extended


https://reviews.llvm.org/D28689





More information about the llvm-commits mailing list