[PATCH] ARM: Align functions containing Thumb-2 jump tables to 4 bytes.

Renato Golin renato.golin at linaro.org
Fri May 1 08:58:40 PDT 2015


With the comment, LGTM. Thanks!


================
Comment at: test/CodeGen/ARM/thumb-alignment.ll:25
@@ +24,3 @@
+; CHECK: .globl	baz
+; CHECK-NEXT: .align	2
+define i32 @baz() {
----------------
Maybe add a check for ADR, to make sure you're not aligning for other reason (and to keep it clear why you're aligning to 4-bytes)?

http://reviews.llvm.org/D9424

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






More information about the llvm-commits mailing list