[all-commits] [llvm/llvm-project] 950061: [NFC] Change a reference member to pointer

Paul T Robinson via All-commits all-commits at lists.llvm.org
Fri Oct 6 13:00:18 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9500616a049995c0d23e070534a41b2ddbec1495
      https://github.com/llvm/llvm-project/commit/9500616a049995c0d23e070534a41b2ddbec1495
  Author: Paul Robinson <paul.robinson at sony.com>
  Date:   2023-10-06 (Fri, 06 Oct 2023)

  Changed paths:
    M clang/lib/Frontend/PrintPreprocessedOutput.cpp

  Log Message:
  -----------
  [NFC] Change a reference member to pointer

This will allow the raw_ostream to be redirected in a subsequent commit.


  Commit: 71d83bb426104fb77605382c61968aaf55b537b8
      https://github.com/llvm/llvm-project/commit/71d83bb426104fb77605382c61968aaf55b537b8
  Author: Paul Robinson <paul.robinson at sony.com>
  Date:   2023-10-06 (Fri, 06 Oct 2023)

  Changed paths:
    M clang/docs/CommandGuide/clang.rst
    M clang/docs/ReleaseNotes.rst
    M clang/include/clang/Basic/DiagnosticDriverKinds.td
    M clang/include/clang/Driver/Options.td
    M clang/include/clang/Frontend/PreprocessorOutputOptions.h
    M clang/lib/Driver/ToolChains/Clang.cpp
    M clang/lib/Frontend/PrintPreprocessedOutput.cpp
    A clang/test/Frontend/Inputs/dashE/dashE.h
    A clang/test/Frontend/Inputs/dashE/sys/a.h
    A clang/test/Frontend/Inputs/dashE/sys/b.h
    A clang/test/Frontend/dashE-sysincludes.cpp
    M clang/test/Preprocessor/minimize-whitespace-messages.c

  Log Message:
  -----------
  Add -fkeep-system-includes modifier for -E

This option will cause -E to preserve the #include directives
for system headers, rather than expanding them into the output.
This can greatly reduce the volume of preprocessed source text
in a test case, making test case reduction simpler.

Note that -fkeep-system-includes is not always appropriate. For
example, if the problem you want to reproduce is induced by a
system header file, it's better to expand those headers fully.
If your source defines symbols that influence the content of a
system header (e.g., _POSIX_SOURCE) then -E will eliminate the
definition, potentially changing the meaning of the preprocessed
source. If you use -isystem to point to non-system headers, for
example to suppress warnings in third-party software, those will
not be expanded and might make the preprocessed source less useful
as a test case.


Compare: https://github.com/llvm/llvm-project/compare/0dfb5dadc6df...71d83bb42610


More information about the All-commits mailing list