[PATCH] D96090: [analyzer] Replace StoreManager::CastRetrievedVal with SValBuilder::evalCast
Denys Petrov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Apr 2 13:17:07 PDT 2021
ASDenysPetrov added a comment.
@steakhal thanks for the approval.
================
Comment at: clang/lib/StaticAnalyzer/Core/SValBuilder.cpp:564-565
}
-
- llvm_unreachable("Unknown SVal kind");
}
----------------
steakhal wrote:
> You probably don't want to remove this. The same applies to similar ones.
Yep. I'd keep it here, beacuse of compiler warnings of no-return function.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96090/new/
https://reviews.llvm.org/D96090
More information about the cfe-commits
mailing list