diff options
| author | Craig Topper <craig.topper@intel.com> | 2018-06-11 16:34:10 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@intel.com> | 2018-06-11 16:34:10 +0000 |
| commit | ae7179414d0dc5404c583b08f8d54fc77f7baaf0 (patch) | |
| tree | 6a09146708ba65232a06d9f609a8f030a51fa320 | |
| parent | c8b7a41a00591c4852ac583cf2bc4c7d34b2fded (diff) | |
| download | bcm5719-llvm-ae7179414d0dc5404c583b08f8d54fc77f7baaf0.tar.gz bcm5719-llvm-ae7179414d0dc5404c583b08f8d54fc77f7baaf0.zip | |
[X86] Properly account for the immediate being multiplied by 8 in the immediate range checking for BI__builtin_ia32_psrldqi128 and friends.
The limit was set to 1023 which only up to 127*8. It needs to be 2047 to allow 255*8.
llvm-svn: 334416
| -rw-r--r-- | clang/lib/Sema/SemaChecking.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang/lib/Sema/SemaChecking.cpp b/clang/lib/Sema/SemaChecking.cpp index 8730408c8de..6648ffc13fe 100644 --- a/clang/lib/Sema/SemaChecking.cpp +++ b/clang/lib/Sema/SemaChecking.cpp @@ -2911,7 +2911,7 @@ bool Sema::CheckX86BuiltinFunctionCall(unsigned BuiltinID, CallExpr *TheCall) { case X86::BI__builtin_ia32_psrldqi128: case X86::BI__builtin_ia32_psrldqi256: case X86::BI__builtin_ia32_psrldqi512: - i = 1; l = 0; u = 1023; + i = 1; l = 0; u = 2047; break; } return SemaBuiltinConstantArgRange(TheCall, i, l, u); |
