dnrops
/
clang
mirror of https://github.com/microsoft/clang.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Use APSInt::isSameValue instead of operator== in a place where two APSInt's 

may have different sizes.  Fixes PR22017


git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@225488 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Richard Trieu 2015-01-09 00:58:16 +00:00
parent 1a9d3540ba
commit a1712b44df
2 changed files with 16 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lib/AST/TemplateBase.cpp
View File

@ -42,7 +42,11 @@ static void printIntegral(const TemplateArgument &TemplArg,
if (const EnumType *ET = T->getAs<EnumType>()) {
for (const EnumConstantDecl* ECD : ET->getDecl()->enumerators()) {
if (ECD->getInitVal() == Val) {
// In Sema::CheckTemplateArugment, enum template arguments value are
// extended to the size of the integer underlying the enum type. This
// may create a size difference between the enum value and template
// argument value, requiring isSameValue here instead of operator==.
if (llvm::APSInt::isSameValue(ECD->getInitVal(), Val)) {
ECD->printQualifiedName(Out, Policy);
return;
}

11
test/SemaTemplate/enum-bool.cpp Normal file
View File

@ -0,0 +1,11 @@
// %RUN: %clang_cc1 -std=c++11 -emit-llvm %s -o %t
enum E : bool { A };
template <E>
struct S {
struct Inner {
Inner() {}
};
};
template class S<A>;