[PATCH] ARM: Do not enforce 4-byte alignment on Thumb-2 functions.

Renato Golin renato.golin at linaro.org
Wed Apr 22 11:06:19 PDT 2015


================
Comment at: lib/Target/ARM/ARMConstantIslandPass.cpp:416
@@ -415,3 @@
-  // Thumb1 functions containing constant pools get 4-byte alignment.
-  // This is so we can keep exact track of where the alignment padding goes.
-
----------------
This is not just about standard Thumb alignment, but Thumb functions with constant pools, which are also expected to align 4, if memory serves me right. Is that what you're testing down there on your new test?

I wonder if that's going to open an old corner case...

http://reviews.llvm.org/D9138

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






More information about the llvm-commits mailing list