diff options
author | Gabor Greif <ggreif@gmail.com> | 2010-09-09 10:51:37 +0000 |
---|---|---|
committer | Gabor Greif <ggreif@gmail.com> | 2010-09-09 10:51:37 +0000 |
commit | 89b06584402a38933e108b66ded3a168cd492dff (patch) | |
tree | d7f7a4efb604c3db57235f4d8841a62f81a1c926 /lib/Checker/ReturnPointerRangeChecker.cpp | |
parent | 96ebad66c451d79c9f57b1edb31efaeeb23b9a01 (diff) |
do not bind temporaries to non-const references
this fixes all analyser test failures in my gcc34-based
environment
how the cast result could bind to the non-const ref is
somewhat mysterious and remains to be investigated; to
avoid similar miscompilations (by gcc34 only?)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@113480 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Checker/ReturnPointerRangeChecker.cpp')
-rw-r--r-- | lib/Checker/ReturnPointerRangeChecker.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/Checker/ReturnPointerRangeChecker.cpp b/lib/Checker/ReturnPointerRangeChecker.cpp index a9eb5ce1a7..0e7a5cf9d1 100644 --- a/lib/Checker/ReturnPointerRangeChecker.cpp +++ b/lib/Checker/ReturnPointerRangeChecker.cpp @@ -59,7 +59,7 @@ void ReturnPointerRangeChecker::PreVisitReturnStmt(CheckerContext &C, if (!ER) return; - DefinedOrUnknownSVal &Idx = cast<DefinedOrUnknownSVal>(ER->getIndex()); + DefinedOrUnknownSVal Idx = cast<DefinedOrUnknownSVal>(ER->getIndex()); // FIXME: All of this out-of-bounds checking should eventually be refactored // into a common place. |