[PATCH] [ARM] Fix large stack alignment codegen bug for ARM and Thumb2 targets

Charlie Turner charlie.turner at arm.com
Wed Jan 7 05:35:05 PST 2015


One other coding-style issue.


================
Comment at: lib/Target/ARM/ARMFrameLowering.cpp:231
@@ +230,3 @@
+  const unsigned AlignMask = Alignment - 1;
+  const unsigned nrBitsToZero = countTrailingZeros(Alignment);
+  assert(!AFI->isThumb1OnlyFunction() && "Thumb1 not supported");
----------------
Coding style: should be `NrBitsToZero`

http://reviews.llvm.org/D6844

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






More information about the llvm-commits mailing list