[clang] [clang-tools-extra] [llvm] Dont alter cold function alignment unless using Os (PR #72387)

Paul T Robinson via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 21 07:01:38 PST 2023


pogo59 wrote:

The psABI doesn't say anything about function alignment, implying that align(1) is functionally correct, and anything more than that is either an optimization or functional dependency for our target. So, if we depend on anything more, we have to set alignment explicitly.

@FlameTop does that explanation sound reasonable to you? So PS4/PS5 should be explicitly setting alignment on all functions that don't have an alignment specified in source. No need for a new attribute.

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


More information about the llvm-commits mailing list