diff options
| author | David Tweed <david.tweed@arm.com> | 2013-01-10 09:11:33 +0000 |
|---|---|---|
| committer | David Tweed <david.tweed@arm.com> | 2013-01-10 09:11:33 +0000 |
| commit | f20318f9e27a064bdd9a3fbb51d2c3a4a4f3ac2e (patch) | |
| tree | 69f7e60ba7609f274457f99f64d51fb22abbeb07 | |
| parent | f9c21665faa7e4936d484396469ee7afc46e1f00 (diff) | |
Testing with a full OpenCL compiler (based on clang) reveals r71734 missed
difference between type widths of a vector and the width of one of its elements
in the case of vector shifts. Use correct witdth in the vector case.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@172047 91177308-0d34-0410-b5e6-96231b3b80d8
| -rw-r--r-- | lib/CodeGen/CGExprScalar.cpp | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/lib/CodeGen/CGExprScalar.cpp b/lib/CodeGen/CGExprScalar.cpp index ae7518a2b9..fb5f58ddbb 100644 --- a/lib/CodeGen/CGExprScalar.cpp +++ b/lib/CodeGen/CGExprScalar.cpp @@ -2368,8 +2368,12 @@ Value *ScalarExprEmitter::EmitSub(const BinOpInfo &op) { } Value *ScalarExprEmitter::GetWidthMinusOneValue(Value* LHS,Value* RHS) { - unsigned Width = cast<llvm::IntegerType>(LHS->getType())->getBitWidth(); - return llvm::ConstantInt::get(RHS->getType(), Width - 1); + llvm::IntegerType *Ty; + if (llvm::VectorType *VT = dyn_cast<llvm::VectorType>(LHS->getType())) + Ty = cast<llvm::IntegerType>(VT->getElementType()); + else + Ty = cast<llvm::IntegerType>(LHS->getType()); + return llvm::ConstantInt::get(RHS->getType(), Ty->getBitWidth() - 1); } Value *ScalarExprEmitter::EmitShl(const BinOpInfo &Ops) { |