[PATCH] D65000: [ARM] Set default alignment to 64bits

Peter Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 22 09:32:33 PDT 2019


peter.smith added a comment.

test case missing A8 aside this looks ok to me. Would like to see if there are any comments from the Pacific time zone.



================
Comment at: test/CodeGen/ARM/exception-alignment.cpp:8
+// A16-NEXT: store <2 x i64> <i64 1, i64 2>, <2 x i64>* [[BC]], align 16
+#include <arm_neon.h>
+
----------------
Are you missing the A8 case? presumably:
```
store <2 x i64> <i64 1, i64 2>, <2 x i64>* [[BC]], align 8
```



Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65000/new/

https://reviews.llvm.org/D65000





More information about the cfe-commits mailing list