[all-commits] [llvm/llvm-project] fd4680: [libc++] Include skipped headers in the --trace-in...

Louis Dionne via All-commits all-commits at lists.llvm.org
Thu Sep 29 12:07:38 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: fd4680fd2cbf0b910e99009a21e8515576b922d8
      https://github.com/llvm/llvm-project/commit/fd4680fd2cbf0b910e99009a21e8515576b922d8
  Author: Louis Dionne <ldionne.2 at gmail.com>
  Date:   2022-09-29 (Thu, 29 Sep 2022)

  Changed paths:
    M libcxx/include/__format/format_error.h
    M libcxx/include/typeinfo
    M libcxx/test/libcxx/transitive_includes.sh.cpp
    M libcxx/test/libcxx/transitive_includes/cxx03.csv
    M libcxx/test/libcxx/transitive_includes/cxx11.csv
    M libcxx/test/libcxx/transitive_includes/cxx14.csv
    M libcxx/test/libcxx/transitive_includes/cxx17.csv
    M libcxx/test/libcxx/transitive_includes/cxx20.csv
    M libcxx/test/libcxx/transitive_includes/cxx2b.csv
    M libcxx/test/libcxx/transitive_includes_to_csv.py

  Log Message:
  -----------
  [libc++] Include skipped headers in the --trace-includes output

By default, Clang does not include headers that are skipped due to
the include guard optimization in the --trace-includes output, which
breaks the use case that we were trying to use it for.

However, Clang does support the -fshow-skipped-includes flag, which
does exactly what we need and will result in an accurate include
graph.

As a fly-by fix, make sure that our includes don't differ between
-fexceptions and -fno-exceptions.

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




More information about the All-commits mailing list