[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