[all-commits] [llvm/llvm-project] 3ea5df: [clang][modules] Only avoid pruning module maps wh...

Jan Svoboda via All-commits all-commits at lists.llvm.org
Fri Apr 19 12:46:01 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 3ea5dff0efdbb4bf5590d882810aa42f9ec26e4e
      https://github.com/llvm/llvm-project/commit/3ea5dff0efdbb4bf5590d882810aa42f9ec26e4e
  Author: Jan Svoboda <jan_svoboda at apple.com>
  Date:   2024-04-19 (Fri, 19 Apr 2024)

  Changed paths:
    M clang/include/clang/Driver/Options.td
    M clang/include/clang/Lex/HeaderSearchOptions.h
    M clang/lib/Serialization/ASTWriter.cpp
    M clang/lib/Tooling/DependencyScanning/ModuleDepCollector.cpp
    M clang/test/ClangScanDeps/modules-full.cpp
    R clang/test/Modules/add-remove-irrelevant-module-map.m
    A clang/test/Modules/prune-non-affecting-module-map-files.m

  Log Message:
  -----------
  [clang][modules] Only avoid pruning module maps when asked to (#89428)

Pruning non-affecting module maps is useful even when passing module
maps explicitly via `-fmodule-map-file=<path>`. For this situation, this
patch reinstates the behavior we had prior to #87849. For the situation
where the explicit module map file arguments were generated by the
dependency scanner (which already pruned the non-affecting ones), this
patch introduces new `-cc1` flag
`-fno-modules-prune-non-affecting-module-map-files` that avoids the
extra work.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list