[PATCH] Fix for 9807. Avoid over-relaxation of 8-bit immediates in integer arithmetic instructions.

David Majnemer david.majnemer at gmail.com
Mon Jun 29 13:03:08 PDT 2015


================
Comment at: lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp:247
@@ +246,3 @@
+  //
+  unsigned relaxableOp = Inst.getNumOperands() - 1;
+  if (Inst.getOperand(relaxableOp).isExpr()) {
----------------
Please name this `RelaxableOp` to match the uppercase convention of the surrounding code.

http://reviews.llvm.org/D10766

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list