[PATCH] D96090: [analyzer] Replace StoreManager::CastRetrievedVal with SValBuilder::evalCast

Denys Petrov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 5 15:19:42 PST 2021


ASDenysPetrov added a comment.

In D96090#2544477 <https://reviews.llvm.org/D96090#2544477>, @steakhal wrote:

> Please, consider removing D95799 <https://reviews.llvm.org/D95799> from the parent revisions if that is not a hard requirement of this patch.

Actually D95799 <https://reviews.llvm.org/D95799> is one of the steps to clean up `CastRetrievedVal` initially. It significantly reduce the next effort of replacing with `evalCast`. But if float cast fix blocks the patch stack, I can try to find a way to bypass it.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96090/new/

https://reviews.llvm.org/D96090



More information about the cfe-commits mailing list