[PATCH] D74447: [Clang] After integrated-cc1, ignore -disable-free when there are more than one job in the queue
Alexandre Ganea via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Feb 12 07:55:42 PST 2020
aganea updated this revision to Diff 244172.
aganea marked 7 inline comments as done.
aganea added a comment.
Address @hans' comments.
While it'd be good to have a bot running without `-disable-free`, I concur with @thakis. This patch will probably introduce too much (untested) variability for release 10.0, if merged.
Let's enable `integrate-cc1` just for one TU for the 10.0, and then we'll see if we want to land this patch or not in the trunk for the 11.0.
I'll open another review.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D74447/new/
https://reviews.llvm.org/D74447
Files:
clang/include/clang/Driver/Job.h
clang/lib/Driver/Driver.cpp
clang/lib/Driver/Job.cpp
clang/lib/Driver/ToolChains/Clang.cpp
clang/test/Driver/cc1-spawnprocess.c
clang/tools/driver/cc1_main.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D74447.244172.patch
Type: text/x-patch
Size: 7306 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200212/29ed76fe/attachment-0001.bin>
More information about the cfe-commits
mailing list