[clang] [llvm] [clang backend] In AArch64's DataLayout, specify a minimum function alignment of 4. (PR #90702)

Doug Wyatt via llvm-commits llvm-commits at lists.llvm.org
Wed May 1 07:56:56 PDT 2024


dougsonos wrote:

> I'd expect regression tests for the autoupgrade, and a test that clang isn't generating the alignment markings anymore.

I just pushed:
- the `-Fn32` is at the end of the data layout strings
- the 6 tests that failed for me locally (testing only AArch64) are fixed

That includes a change to CodeGen/CodeGenCXX/member-alignment.cpp, which checks for alignment / no alignment on C++ methods, depending on target.

I'm not clear on when the autoupgrade runs and how to test it. Is there an example I can follow?

Thanks!

https://github.com/llvm/llvm-project/pull/90702


More information about the llvm-commits mailing list