[llvm] [X86] Fix typo: QWORD alignment is greater than 4, not 8 (PR #87819)
Craig Topper via llvm-commits
llvm-commits at lists.llvm.org
Fri Apr 5 12:14:01 PDT 2024
topperc wrote:
> > But no tests are affected?
>
> Align(8) is QWORD aligned, but this was checking to see f alignment was higher than that, which it is not.
>
> Another PR I am working on for optsize depends on this, but because it is an oversight or typo that affects the compiler, I figure I would make this into a separate PR.
Your description said "This probably inhibited a lot of optimizations.". That should be pretty easy to prove so why no test updates?
https://github.com/llvm/llvm-project/pull/87819
More information about the llvm-commits
mailing list