[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