[PATCH] D61228: [PowerPC] Set the innermost hot loop(from PGO) to align 32 bytes
Nemanja Ivanovic via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 16 14:29:44 PDT 2019
nemanjai added a comment.
> Can you explain why we don't always do this? You're checking for profiling data but then not using it?
I think the idea is that with PGO data, we are more certain that the hotness information is actually meaningful - since the alignment directive will only be emitted for loops that are "hot". Without PGO data, we will align a majority of loops which may be overkill. But this should be clearly stated in the comment though.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61228/new/
https://reviews.llvm.org/D61228
More information about the llvm-commits
mailing list