[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