[all-commits] [llvm/llvm-project] 5b3b00: Move llvm::array_lenghtof to llvm/ADT/STLArrayExtr...

serge-sans-paille via All-commits all-commits at lists.llvm.org
Fri Jan 28 02:51:18 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 5b3b008cf07df2b6ae8b0f49b6557dee7ad29262
      https://github.com/llvm/llvm-project/commit/5b3b008cf07df2b6ae8b0f49b6557dee7ad29262
  Author: serge-sans-paille <sguelton at redhat.com>
  Date:   2022-01-28 (Fri, 28 Jan 2022)

  Changed paths:
    A llvm/include/llvm/ADT/STLArrayExtras.h
    M llvm/include/llvm/ADT/STLExtras.h
    M llvm/lib/Support/ARMAttributeParser.cpp
    M llvm/lib/Support/Signals.cpp
    M llvm/lib/Support/Triple.cpp
    M llvm/lib/Support/raw_ostream.cpp

  Log Message:
  -----------
  Move llvm::array_lenghtof to llvm/ADT/STLArrayExtras.h

This moves the dependency of several files on include/llvm/ADT/STLExtras.h to
the much shorter llvm/ADT/STLArrayExtras.h

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


  Commit: b2e0a6d95a261563b166694ef8bb92110376c0c5
      https://github.com/llvm/llvm-project/commit/b2e0a6d95a261563b166694ef8bb92110376c0c5
  Author: serge-sans-paille <sguelton at redhat.com>
  Date:   2022-01-28 (Fri, 28 Jan 2022)

  Changed paths:
    M llvm/include/llvm/Bitstream/BitstreamReader.h

  Log Message:
  -----------
  Cleanup includes for LLVMBitstream

This is a very small library, so only a few changes and very little gain.

$ clang++ -E  -Iinclude -I../llvm/include ../llvm/lib/Bitstream/Reader/*.cpp -std=c++14 -fno-rtti -fno-exceptions | wc -l
before: 65777
after:  64997

Related Discourse thread: https://llvm.discourse.group/t/include-what-you-use-include-cleanup

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


  Commit: 92f49b89e2d8aaa7826d96ec2d45dfe1102371b9
      https://github.com/llvm/llvm-project/commit/92f49b89e2d8aaa7826d96ec2d45dfe1102371b9
  Author: serge-sans-paille <sguelton at redhat.com>
  Date:   2022-01-28 (Fri, 28 Jan 2022)

  Changed paths:
    M llvm/lib/TableGen/DetailedRecordsBackend.cpp
    M llvm/lib/TableGen/JSONBackend.cpp
    M llvm/lib/TableGen/Main.cpp
    M llvm/lib/TableGen/Record.cpp
    M llvm/lib/TableGen/SetTheory.cpp
    M llvm/lib/TableGen/TGParser.cpp
    M llvm/lib/TableGen/TableGenBackendSkeleton.cpp

  Log Message:
  -----------
  Cleanup include dependencies for LLVMTableGen

Based on the output of include-what-you-use. No other library seems affected by
the new forward declaration.

$ clang++ -E  -Iinclude -I../llvm/include ../llvm/lib/TableGen/*.cpp -std=c++14 -fno-rtti -fno-exceptions | wc -l
before: 795231
after: 750654

Related Discourse thread: https://llvm.discourse.group/t/include-what-you-use-include-cleanup
Differential Revision: https://reviews.llvm.org/D118374


  Commit: bd059b3bc97aa15904e93cccc983ad638ad2310e
      https://github.com/llvm/llvm-project/commit/bd059b3bc97aa15904e93cccc983ad638ad2310e
  Author: serge-sans-paille <sguelton at redhat.com>
  Date:   2022-01-28 (Fri, 28 Jan 2022)

  Changed paths:
    M llvm/lib/TableGen/TableGenBackendSkeleton.cpp
    M llvm/utils/TableGen/GICombinerEmitter.cpp
    M llvm/utils/TableGen/GlobalISel/CodeExpander.cpp
    M llvm/utils/TableGen/GlobalISel/CodeExpander.h
    M llvm/utils/TableGen/GlobalISel/GIMatchDag.h
    M llvm/utils/TableGen/GlobalISel/GIMatchDagEdge.cpp
    M llvm/utils/TableGen/GlobalISel/GIMatchDagInstr.h
    M llvm/utils/TableGen/GlobalISel/GIMatchDagPredicate.cpp
    M llvm/utils/TableGen/GlobalISel/GIMatchDagPredicate.h
    M llvm/utils/TableGen/GlobalISel/GIMatchDagPredicateDependencyEdge.cpp
    M llvm/utils/TableGen/GlobalISel/GIMatchDagPredicateDependencyEdge.h
    M llvm/utils/TableGen/GlobalISel/GIMatchTree.cpp

  Log Message:
  -----------
  Cleanup includes for LLVMTableGenGlobalISel

Based on the output of include-what-you-use. No big deal here, it's a utils
library and it doesn't seem to be used a lot across the codebase.

$ clang++ -E  -Iinclude -I../llvm/include ../llvm/utils/TableGen/GlobalISel/*.cpp -std=c++14 -fno-rtti -fno-exceptions | wc -l
before: 573143
after:  568908

Related Discourse thread: https://llvm.discourse.group/t/include-what-you-use-include-cleanup
Differential Revision: https://reviews.llvm.org/D118375


Compare: https://github.com/llvm/llvm-project/compare/b339bbdb1970...bd059b3bc97a


More information about the All-commits mailing list