lxfind added a comment.
Herald added subscribers: manas, steakhal, ASDenysPetrov, martong, dkrupp,
donat.nagy, Szelethus, mikhail.ramalho, baloghadamsoftware.
Hi! I have a question regarding the implementation of
"VisitMaterializeTemporaryExpr". Specifically, I wonder if we should skip
This revision was automatically updated to reflect the committed changes.
Closed by commit rC326014: [CFG] Provide construction contexts for
lifetime-extended temporaries. (authored by dergachev, committed by ).
Repository:
rC Clang
https://reviews.llvm.org/D43477
Files:
This revision was automatically updated to reflect the committed changes.
Closed by commit rL326014: [CFG] Provide construction contexts for
lifetime-extended temporaries. (authored by dergachev, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
dcoughlin accepted this revision.
dcoughlin added a comment.
This revision is now accepted and ready to land.
LGTM.
Repository:
rC Clang
https://reviews.llvm.org/D43477
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
NoQ added a comment.
> eg. `const C (123);` or the actual (not the elidable copy) constructor in
> `C foo() { return C(123); }`
Emm, sry, never mind, forget it, i was trying to say that the reason why we
don't have a `CXXBindTemporary` is because we don't have a destructor in class
`C`, not
NoQ created this revision.
NoQ added reviewers: dcoughlin, xazax.hun, a.sidorin, george.karpenkov, szepet.
Herald added subscribers: cfe-commits, rnkovacs.
`MaterializeTemporaryExpr` captures lifetime extension information. In the
analyzer it is important to have this information at construction
