[PATCH] D66715: [CFG] Add dumps for CFGElement and CFGElementRef

2019-09-12 Thread Kristóf Umann via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL371765: [CFG] Add dumps for CFGElement and CFGElementRef (authored by Szelethus, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://r

[PATCH] D66715: [CFG] Add dumps for CFGElement and CFGElementRef

2019-08-28 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Thanks!! In D66715#1644395 , @Szelethus wrote: > Mind that I'll probably commit without the unit test, last time I learned the > hard way that the AST's lif

[PATCH] D66715: [CFG] Add dumps for CFGElement and CFGElementRef

2019-08-25 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus added a comment. Mind that I'll probably commit without the unit test, last time I learned the hard way that the AST's lifetime ends by the time we retrieve the CFG. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66715/new/ https://review

[PATCH] D66715: [CFG] Add dumps for CFGElement and CFGElementRef

2019-08-25 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus created this revision. Szelethus added reviewers: NoQ, xazax.hun, dcoughlin, rnkovacs, Charusso, baloghadamsoftware. Szelethus added a project: clang. Herald added subscribers: cfe-commits, gamesh411, dkrupp. Seems like we never had these, so here we go! I also did some refactoring as I