[PATCH] D12336: x32. Fixes a bug in i8mem_NOREX declaration.

Derek Schuff via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 31 16:10:34 PDT 2015


dschuff added a subscriber: dschuff.

================
Comment at: lib/Target/X86/X86InstrInfo.td:344
@@ +343,3 @@
+// of a plain GPR, so that it doesn't potentially require a REX prefix.
+def ptr_rc_norex : PointerLikeRegClass<3>;
+def ptr_rc_norex_nosp : PointerLikeRegClass<4>;
----------------
Maybe we should order these kind numbers in the order they appear in the file (i.e. these are 1 and 2, and ptr_rc_tailcall becomes 3)

================
Comment at: lib/Target/X86/X86RegisterInfo.cpp:171
@@ -170,3 +170,3 @@
     return &X86::GR32_NOSPRegClass;
-  case 2: // Available for tailcall (not callee-saved GPRs).
+  case 2: {// Available for tailcall (not callee-saved GPRs).
     const Function *F = MF.getFunction();
----------------
space between { and //


http://reviews.llvm.org/D12336





More information about the llvm-commits mailing list