[PATCH] D158977: [clang][dataflow] Don't associate prvalue expressions with storage locations.
Gábor Horváth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 28 19:26:46 PDT 2023
xazax.hun accepted this revision.
xazax.hun added a comment.
This revision is now accepted and ready to land.
Thanks! Sometimes I am wondering whether we actually need a full map for PRValues. E.g., once we processed a `MaterializeTemporaryExpr`, we now have a location for the value, and it feels like we represent the same thing twice, once in `ExprToLoc + LocToVal` and once in `ExprToVal`. It is probably not too bad and might be extra work to clean this up.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D158977/new/
https://reviews.llvm.org/D158977
More information about the cfe-commits
mailing list