diff options
| author | Balazs Benics <balazs.benics@sigmatechnology.se> | 2021-10-20 17:43:31 +0200 |
|---|---|---|
| committer | Balazs Benics <balazs.benics@sigmatechnology.se> | 2021-10-20 17:43:31 +0200 |
| commit | 16be17ad4b13deb23c893c3c668076e70b07f289 (patch) | |
| tree | fc38eb70f938fbafa43b0b6a0e7254173e8dbccc /clang/lib/FrontendTool/ExecuteCompilerInvocation.cpp | |
| parent | aab0f2264aebe6cb89d5dcb30df664bfc4d8f1c3 (diff) | |
| download | llvm-16be17ad4b13deb23c893c3c668076e70b07f289.zip llvm-16be17ad4b13deb23c893c3c668076e70b07f289.tar.gz llvm-16be17ad4b13deb23c893c3c668076e70b07f289.tar.bz2 | |
[analyzer][NFC] Refactor llvm::isa<> usages in the StaticAnalyzer
It turns out llvm::isa<> is variadic, and we could have used this at a
lot of places.
The following patterns:
x && isa<T1>(x) || isa<T2>(x) ...
Will be replaced by:
isa_and_non_null<T1, T2, ...>(x)
Sometimes it caused further simplifications, when it would cause even
more code smell.
Aside from this, keep in mind that within `assert()` or any macro
functions, we need to wrap the isa<> expression within a parenthesis,
due to the parsing of the comma.
Reviewed By: martong
Differential Revision: https://reviews.llvm.org/D111982
Diffstat (limited to 'clang/lib/FrontendTool/ExecuteCompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions