diff options
| author | Aaron Ballman <aaron@aaronballman.com> | 2014-01-16 14:32:03 +0000 |
|---|---|---|
| committer | Aaron Ballman <aaron@aaronballman.com> | 2014-01-16 14:32:03 +0000 |
| commit | 4755304fd1abd54499867c306458df2782fb9252 (patch) | |
| tree | a7bb11fd1698b807f15d42cf703228da941bb4f7 /clang/utils/TableGen | |
| parent | 980589a8035588ddfd7bb92b0adee4ccfee4ece2 (diff) | |
| download | bcm5719-llvm-4755304fd1abd54499867c306458df2782fb9252.tar.gz bcm5719-llvm-4755304fd1abd54499867c306458df2782fb9252.zip | |
Fixing a warning that causes the sanitizer build disliked about mixing && and ||. Since this is generated code, the && has been removed from the expression entirely.
llvm-svn: 199392
Diffstat (limited to 'clang/utils/TableGen')
| -rw-r--r-- | clang/utils/TableGen/ClangAttrEmitter.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/clang/utils/TableGen/ClangAttrEmitter.cpp b/clang/utils/TableGen/ClangAttrEmitter.cpp index 672589c0b48..7ac14ba915b 100644 --- a/clang/utils/TableGen/ClangAttrEmitter.cpp +++ b/clang/utils/TableGen/ClangAttrEmitter.cpp @@ -2088,10 +2088,11 @@ static std::string GenerateCustomAppertainsTo(const Record &Subject, } OS << "static bool " << FnName << "(const Decl *D) {\n"; - OS << " const " << GetSubjectWithSuffix(Base) << " *S = dyn_cast<"; + OS << " if (const " << GetSubjectWithSuffix(Base) << " *S = dyn_cast<"; OS << GetSubjectWithSuffix(Base); - OS << ">(D);\n"; - OS << " return S && " << Subject.getValueAsString("CheckCode") << ";\n"; + OS << ">(D))\n"; + OS << " return " << Subject.getValueAsString("CheckCode") << ";\n"; + OS << " return false;\n"; OS << "}\n\n"; CustomSubjectSet.insert(FnName); |
