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

Rafael EspĂ­ndola rafael.espindola at gmail.com
Mon Jun 29 13:05:40 PDT 2015


On 29 June 2015 at 16:03, David Majnemer <david.majnemer at gmail.com> wrote:
> ================
> 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.

Oops, thanks for catching this.



More information about the llvm-commits mailing list