[PATCH] D46393: Remove explicit cfg-temporary-dtors=true

2018-05-04 Thread Alexander Kornienko via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL331520: Remove explicit cfg-temporary-dtors=true (authored by alexfh, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/D46393 Files: cfe/trunk/te

[PATCH] D46393: Remove explicit cfg-temporary-dtors=true

2018-05-04 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh added a comment. In https://reviews.llvm.org/D46393#1086887, @NoQ wrote: > Thanks! > > Just curious - did these flags bother you? Cause we never really care about > cleaning up run lines after flipping the flag, so we have a lot of such stale > flags in our tests. We could start cleaning

[PATCH] D46393: Remove explicit cfg-temporary-dtors=true

2018-05-03 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! Just curious - did these flags bother you? Cause we never really care about cleaning up run lines after flipping the flag, so we have a lot of such stale flags in our tests. We could start

[PATCH] D46393: Remove explicit cfg-temporary-dtors=true

2018-05-03 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh created this revision. alexfh added a reviewer: NoQ. Remove explicit -analyzer-config cfg-temporary-dtors=true in analyzer tests, since this option defaults to true since r326461. Repository: rC Clang https://reviews.llvm.org/D46393 Files: test/Analysis/cfg-rich-constructors.cpp t