[PATCH] D114536: [X86][MS] Fix the wrong alignment of vector variable arguments on Win32
Reid Kleckner via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 2 10:22:51 PST 2022
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
Sorry for the delay, I was out sick, and this fell out of my inbox.
================
Comment at: llvm/lib/Target/X86/X86ISelLowering.cpp:4094
+ MaybeAlign Alignment;
+ if (Subtarget.isTargetWindowsMSVC() && !Subtarget.is64Bit() &&
+ Arg.getSimpleValueType() != MVT::f80)
----------------
I think this could be simplified to use getStackAlign, but I won't insist.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D114536/new/
https://reviews.llvm.org/D114536
More information about the llvm-commits
mailing list