[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