diff options
author | vabridgers <58314289+vabridgers@users.noreply.github.com> | 2023-09-20 06:11:39 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-09-20 06:11:39 -0500 |
commit | da26500aa82316f607ddc60e56982775d7f45cff (patch) | |
tree | da767449a860b68b6fb113d20af51b783814495c /llvm/lib/IR | |
parent | 522c1d0eeaa272381ade1af8b9ce9a9ab9180ea3 (diff) | |
download | llvm-da26500aa82316f607ddc60e56982775d7f45cff.zip llvm-da26500aa82316f607ddc60e56982775d7f45cff.tar.gz llvm-da26500aa82316f607ddc60e56982775d7f45cff.tar.bz2 |
[analyzer] Fix crash analyzing _BitInt() in evalIntegralCast (#66782)
evalIntegralCast was using makeIntVal, and when _BitInt() types were
introduced this exposed a crash in evalIntegralCast as a result.
This is a reapply of a previous patch that failed post merge on the arm
buildbots, because arm cannot handle large
BitInts. Pinning the triple for the testcase solves that problem.
Improve evalIntegralCast to use makeIntVal more efficiently to avoid the
crash exposed by use of _BitInt.
This was caught with our internal randomized testing.
<src-root>/llvm/include/llvm/ADT/APInt.h:1510:
int64_t llvm::APInt::getSExtValue() const: Assertion
`getSignificantBits() <= 64 && "Too many bits for int64_t"' failed.a
...
#9 <address> llvm::APInt::getSExtValue() const
<src-root>/llvm/include/llvm/ADT/APInt.h:1510:5
llvm::IntrusiveRefCntPtr<clang::ento::ProgramState const>,
clang::ento::SVal, clang::QualType, clang::QualType)
<src-root>/clang/lib/StaticAnalyzer/Core/SValBuilder.cpp:607:24
clang::Expr const*, clang::ento::ExplodedNode*,
clang::ento::ExplodedNodeSet&)
<src-root>/clang/lib/StaticAnalyzer/Core/ExprEngineC.cpp:413:61
...
Fixes: https://github.com/llvm/llvm-project/issues/61960
Reviewed By: donat.nagy
Diffstat (limited to 'llvm/lib/IR')
0 files changed, 0 insertions, 0 deletions