diff options
author | Daniel Dunbar <daniel@zuster.org> | 2008-10-24 08:07:57 +0000 |
---|---|---|
committer | Daniel Dunbar <daniel@zuster.org> | 2008-10-24 08:07:57 +0000 |
commit | d7be95d60a098e4643cf049309ce1f51806cd781 (patch) | |
tree | 7455a5125f74d6054a30a391af9a26f1e0eb60e8 /clang/lib/AST/ExprConstant.cpp | |
parent | b2d68310cb81c31e721e8dbb74d9713430147edb (diff) | |
download | bcm5719-llvm-d7be95d60a098e4643cf049309ce1f51806cd781.tar.gz bcm5719-llvm-d7be95d60a098e4643cf049309ce1f51806cd781.zip |
PR2919: __builtin_types_compatible_p strips CRV qualifiers.
llvm-svn: 58079
Diffstat (limited to 'clang/lib/AST/ExprConstant.cpp')
-rw-r--r-- | clang/lib/AST/ExprConstant.cpp | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/clang/lib/AST/ExprConstant.cpp b/clang/lib/AST/ExprConstant.cpp index 5bc4170e70a..12010646bce 100644 --- a/clang/lib/AST/ExprConstant.cpp +++ b/clang/lib/AST/ExprConstant.cpp @@ -205,7 +205,13 @@ public: } bool VisitTypesCompatibleExpr(const TypesCompatibleExpr *E) { Result.zextOrTrunc(getIntTypeSizeInBits(E->getType())); - Result = Info.Ctx.typesAreCompatible(E->getArgType1(), E->getArgType2()); + // Per gcc docs "this built-in function ignores top level + // qualifiers". We need to use the canonical version to properly + // be able to strip CRV qualifiers from the type. + QualType T0 = Info.Ctx.getCanonicalType(E->getArgType1()); + QualType T1 = Info.Ctx.getCanonicalType(E->getArgType2()); + Result = Info.Ctx.typesAreCompatible(T0.getUnqualifiedType(), + T1.getUnqualifiedType()); return true; } bool VisitDeclRefExpr(const DeclRefExpr *E); |