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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
