[llvm] [X86] Fix typo: QWORD alignment is greater than 4, not 8 (PR #87819)

via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 5 13:13:25 PDT 2024


AtariDreams 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 original description said "This probably inhibited a lot of optimizations.". That should be pretty easy to prove so why no test updates?

Done!

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


More information about the llvm-commits mailing list