r281706 - Fix unused result from sign extending an Offset.
Stephen Hines via cfe-commits
cfe-commits at lists.llvm.org
Fri Sep 16 00:21:25 PDT 2016
Author: srhines
Date: Fri Sep 16 02:21:24 2016
New Revision: 281706
URL: http://llvm.org/viewvc/llvm-project?rev=281706&view=rev
Log:
Fix unused result from sign extending an Offset.
Summary:
Offset was doubled in size, but the assignment was missing. We just need
to reassign to the original variable in this case to fix it.
Reviewers: cfe-commits, echristo
Subscribers: meikeb
Differential Revision: https://reviews.llvm.org/D24648
Modified:
cfe/trunk/lib/Sema/SemaChecking.cpp
Modified: cfe/trunk/lib/Sema/SemaChecking.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaChecking.cpp?rev=281706&r1=281705&r2=281706&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaChecking.cpp (original)
+++ cfe/trunk/lib/Sema/SemaChecking.cpp Fri Sep 16 02:21:24 2016
@@ -3882,7 +3882,7 @@ static void sumOffsets(llvm::APSInt &Off
// possible.
if (Ov) {
assert(BitWidth <= UINT_MAX / 2 && "index (intermediate) result too big");
- Offset.sext(2 * BitWidth);
+ Offset = Offset.sext(2 * BitWidth);
sumOffsets(Offset, Addend, BinOpKind, AddendIsRight);
return;
}
More information about the cfe-commits
mailing list