[all-commits] [llvm/llvm-project] 9c9e46: [Clang] Limit -fintegrated-cc1 to only one TU

Alexandre Ganea via All-commits all-commits at lists.llvm.org
Thu Feb 13 02:03:53 PST 2020


  Branch: refs/heads/release/10.x
  Home:   https://github.com/llvm/llvm-project
  Commit: 9c9e46d786d0581079e5018e550cbe187a1ec219
      https://github.com/llvm/llvm-project/commit/9c9e46d786d0581079e5018e550cbe187a1ec219
  Author: Alexandre Ganea <alexandre.ganea at ubisoft.com>
  Date:   2020-02-13 (Thu, 13 Feb 2020)

  Changed paths:
    M clang/include/clang/Driver/Job.h
    M clang/lib/Driver/Driver.cpp
    M clang/lib/Driver/Job.cpp
    M clang/lib/Driver/ToolChains/Clang.cpp
    M clang/test/Driver/cc1-spawnprocess.c

  Log Message:
  -----------
  [Clang] Limit -fintegrated-cc1 to only one TU

As discussed in https://reviews.llvm.org/D74447, this patch disables integrated-cc1 behavior if there's more than one job to be executed. This is meant to limit memory bloating, given that currently jobs don't clean up after execution (-disable-free is always active in cc1 mode).

I see this behavior as temporary until release 10.0 ships (to ease merging of this patch), then we'll reevaluate the situation, see if D74447 makes more sense on the long term.

Differential Revision: https://reviews.llvm.org/D74490

(cherry picked from commit 20f1abe306d030e99f56185a3aa077ffadf59b8a)




More information about the All-commits mailing list