diff options
| author | Hans Wennborg <hans@hanshq.net> | 2018-03-26 10:07:51 +0000 |
|---|---|---|
| committer | Hans Wennborg <hans@hanshq.net> | 2018-03-26 10:07:51 +0000 |
| commit | 311b63f13b0873b183f2a7e57eecf0257c7bffa0 (patch) | |
| tree | 3a063a1198c34d6370c0a3edb1ebb8eadbc4149e /llvm/lib/Target/X86/X86ISelLowering.cpp | |
| parent | 8840f644b40b4f764ff5cd297d454429a2fd63c5 (diff) | |
| download | bcm5719-llvm-311b63f13b0873b183f2a7e57eecf0257c7bffa0.tar.gz bcm5719-llvm-311b63f13b0873b183f2a7e57eecf0257c7bffa0.zip | |
Revert r328386 "[X86] Fix Windows `i1 zeroext` conventions to use i8 instead of i32"
This broke Chromium (see crbug.com/825748). It looks like mstorsjo's follow-up
patch at D44876 fixes this, but let's revert back to green for now until that's
ready to land.
(Also reverts r328443.)
> Both GCC and MSVC only look at the low byte of a boolean when it is
> passed.
llvm-svn: 328482
Diffstat (limited to 'llvm/lib/Target/X86/X86ISelLowering.cpp')
| -rw-r--r-- | llvm/lib/Target/X86/X86ISelLowering.cpp | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp index 8e9e090263e..242a008d050 100644 --- a/llvm/lib/Target/X86/X86ISelLowering.cpp +++ b/llvm/lib/Target/X86/X86ISelLowering.cpp @@ -3034,11 +3034,7 @@ SDValue X86TargetLowering::LowerFormalArguments( getv64i1Argument(VA, ArgLocs[++I], Chain, DAG, dl, Subtarget); } else { const TargetRegisterClass *RC; - if (RegVT == MVT::i8) - RC = &X86::GR8RegClass; - else if (RegVT == MVT::i16) - RC = &X86::GR16RegClass; - else if (RegVT == MVT::i32) + if (RegVT == MVT::i32) RC = &X86::GR32RegClass; else if (Is64Bit && RegVT == MVT::i64) RC = &X86::GR64RegClass; |
