[PATCH] D111270: [clang] Pass -clear-ast-before-backend in Clang::ConstructJob()

2021-10-15 Thread Heejin Ahn via Phabricator via cfe-commits
aheejin added a comment. After this patch, `-print-stats` started erroring out. I filed a bug at https://bugs.llvm.org/show_bug.cgi?id=52193. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111270/new/ https://reviews.llvm.org/D111270

[PATCH] D111270: [clang] Pass -clear-ast-before-backend in Clang::ConstructJob()

2021-10-15 Thread Arthur Eubanks via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG47eb99aa44ab: [clang] Pass -clear-ast-before-backend in Clang::ConstructJob() (authored by aeubanks). Repository: rG LLVM Github Monorepo

[PATCH] D111270: [clang] Pass -clear-ast-before-backend in Clang::ConstructJob()

2021-10-14 Thread Arthur Eubanks via Phabricator via cfe-commits
aeubanks updated this revision to Diff 379866. aeubanks added a comment. force off for the clang interpreter I thought I ran check-clang, apparently I didn't Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111270/new/

[PATCH] D111270: [clang] Pass -clear-ast-before-backend in Clang::ConstructJob()

2021-10-14 Thread David Blaikie via Phabricator via cfe-commits
dblaikie accepted this revision. dblaikie added a comment. This revision is now accepted and ready to land. In D111270#3062802 , @aeubanks wrote: > In D111270#3060484 , @dblaikie > wrote: > >> Plan is still to

[PATCH] D111270: [clang] Pass -clear-ast-before-backend in Clang::ConstructJob()

2021-10-13 Thread Arthur Eubanks via Phabricator via cfe-commits
aeubanks added a comment. In D111270#3060484 , @dblaikie wrote: > Plan is still to address the "this only works with disable free" issue? (I've > some preference that be addressed before this feature is turned on by default)

[PATCH] D111270: [clang] Pass -clear-ast-before-backend in Clang::ConstructJob()

2021-10-13 Thread David Blaikie via Phabricator via cfe-commits
dblaikie added a comment. Plan is still to address the "this only works with disable free" issue? (I've some preference that be addressed before this feature is turned on by default) & is there a flag to pass to turn this off if someone had trouble with it? (doesn't necessarily have to be, but

[PATCH] D111270: [clang] Pass -clear-ast-before-backend in Clang::ConstructJob()

2021-10-12 Thread Arthur Eubanks via Phabricator via cfe-commits
aeubanks created this revision. aeubanks updated this revision to Diff 378327. aeubanks added a comment. aeubanks edited the summary of this revision. aeubanks retitled this revision from "[clang] Always pass -clear-ast-before-backend" to "[clang] Pass -clear-ast-before-backend". aeubanks edited