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

[Paul T Robinson via llvm-commits]

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