diff --git a/clang/include/clang/StaticAnalyzer/Core/PathSensitive/BasicValueFactory.h b/clang/include/clang/StaticAnalyzer/Core/PathSensitive/BasicValueFactory.h --- a/clang/include/clang/StaticAnalyzer/Core/PathSensitive/BasicValueFactory.h +++ b/clang/include/clang/StaticAnalyzer/Core/PathSensitive/BasicValueFactory.h @@ -152,9 +152,13 @@ T = AT->getValueType(); } - assert(T->isIntegralOrEnumerationType() || Loc::isLocType(T)); - return APSIntType(Ctx.getIntWidth(T), - !T->isSignedIntegerOrEnumerationType()); + if (T->isIntegralOrEnumerationType() || Loc::isLocType(T)) { + return APSIntType(Ctx.getIntWidth(T), + !T->isSignedIntegerOrEnumerationType()); + } else if (T->isFixedPointType()) { + return APSIntType(Ctx.getIntWidth(T), !T->isSignedFixedPointType()); + } + llvm_unreachable("Unsupported type in getAPSIntType!"); } /// Convert - Create a new persistent APSInt with the same value as 'From' diff --git a/clang/test/Analysis/fixed-point.c b/clang/test/Analysis/fixed-point.c new file mode 100644 --- /dev/null +++ b/clang/test/Analysis/fixed-point.c @@ -0,0 +1,65 @@ +// RUN: %clang_analyze_cc1 -ffixed-point \ +// RUN: -analyzer-checker=core,debug.ExprInspection -verify %s + +// expected-no-diagnostics + +// Check that getAPSIntType does not crash +// when using fixed point types. + +enum Kind { en_0 = 1 }; + +void _enum(int c) { + (void)((enum Kind) c >> 4); +} + +void _inttype(int c) { + (void)(c >> 4); +} + +void _accum(int c) { + (void)((_Accum) c >> 4); +} + +void _fract(int c) { + (void)((_Fract) c >> 4); +} + +void _long_fract(int c) { + (void)((long _Fract) c >> 4); +} + +void _unsigned_accum(int c) { + (void)((unsigned _Accum) c >> 4); +} + +void _short_unsigned_accum(int c) { + (void)((short unsigned _Accum) c >> 4); +} + +void _unsigned_fract(int c) { + (void)((unsigned _Fract) c >> 4); +} + +void sat_accum(int c) { + (void)((_Sat _Accum) c >> 4); +} + +void sat_fract(int c) { + (void)((_Sat _Fract) c >> 4); +} + +void sat_long_fract(int c) { + (void)((_Sat long _Fract) c >> 4); +} + +void sat_unsigned_accum(int c) { + (void)((_Sat unsigned _Accum) c >> 4); +} + +void sat_short_unsigned_accum(int c) { + (void)((_Sat short unsigned _Accum) c >> 4); +} + +void sat_unsigned_fract(int c) { + (void)((_Sat unsigned _Fract) c >> 4); +} diff --git a/clang/unittests/StaticAnalyzer/APSIntTypeTest.cpp b/clang/unittests/StaticAnalyzer/APSIntTypeTest.cpp new file mode 100644 --- /dev/null +++ b/clang/unittests/StaticAnalyzer/APSIntTypeTest.cpp @@ -0,0 +1,57 @@ +//===- unittest/StaticAnalyzer/APSIntTest.cpp - getAPSIntType test --===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception +// +//===----------------------------------------------------------------------===// + +#include "clang/Basic/TargetInfo.h" +#include "clang/StaticAnalyzer/Core/PathSensitive/RangedConstraintManager.h" +#include "clang/Tooling/Tooling.h" +#include "llvm/ADT/APSInt.h" +#include "llvm/Support/raw_ostream.h" +#include "gtest/gtest.h" + +namespace { + +TEST(getAPSIntTypeTest, APSIntTypeTests) { + std::unique_ptr AST = clang::tooling::buildASTFromCode(""); + clang::ASTContext &Context = AST->getASTContext(); + llvm::BumpPtrAllocator Arena; + clang::ento::BasicValueFactory BVF{Context, Arena}; + + clang::ento::APSIntType Ty = BVF.getAPSIntType(Context.LongAccumTy); + EXPECT_TRUE(Ty.getBitWidth() == Context.getTargetInfo().getLongAccumWidth()); + EXPECT_FALSE(Ty.isUnsigned()); + + Ty = BVF.getAPSIntType(Context.UnsignedLongAccumTy); + EXPECT_TRUE(Ty.getBitWidth() == Context.getTargetInfo().getLongAccumWidth()); + EXPECT_TRUE(Ty.isUnsigned()); + + Ty = BVF.getAPSIntType(Context.LongFractTy); + EXPECT_TRUE(Ty.getBitWidth() == Context.getTargetInfo().getLongFractWidth()); + EXPECT_FALSE(Ty.isUnsigned()); + + Ty = BVF.getAPSIntType(Context.UnsignedLongFractTy); + EXPECT_TRUE(Ty.getBitWidth() == Context.getTargetInfo().getLongFractWidth()); + EXPECT_TRUE(Ty.isUnsigned()); + + Ty = BVF.getAPSIntType(Context.SignedCharTy); + EXPECT_TRUE(Ty.getBitWidth() == Context.getTargetInfo().getCharWidth()); + EXPECT_FALSE(Ty.isUnsigned()); + + Ty = BVF.getAPSIntType(Context.UnsignedCharTy); + EXPECT_TRUE(Ty.getBitWidth() == Context.getTargetInfo().getCharWidth()); + EXPECT_TRUE(Ty.isUnsigned()); + + Ty = BVF.getAPSIntType(Context.LongTy); + EXPECT_TRUE(Ty.getBitWidth() == Context.getTargetInfo().getLongWidth()); + EXPECT_FALSE(Ty.isUnsigned()); + + Ty = BVF.getAPSIntType(Context.UnsignedLongTy); + EXPECT_TRUE(Ty.getBitWidth() == Context.getTargetInfo().getLongWidth()); + EXPECT_TRUE(Ty.isUnsigned()); +} + +} // end namespace diff --git a/clang/unittests/StaticAnalyzer/CMakeLists.txt b/clang/unittests/StaticAnalyzer/CMakeLists.txt --- a/clang/unittests/StaticAnalyzer/CMakeLists.txt +++ b/clang/unittests/StaticAnalyzer/CMakeLists.txt @@ -5,6 +5,7 @@ add_clang_unittest(StaticAnalysisTests AnalyzerOptionsTest.cpp + APSIntTypeTest.cpp BugReportInterestingnessTest.cpp CallDescriptionTest.cpp CallEventTest.cpp