[PATCH] D156710: [TargetParser] Generate RISCVTargetParserDef only if RISCV is enabled.

[Francesco Petrogalli via Phabricator via llvm-commits]

fpetrogalli added a comment.

In D156710#4547365 <https://reviews.llvm.org/D156710#4547365>, @craig.topper wrote:

> Does this improve build times by a significant amount?

I'd say `~0.75x` speedup when building LLVMTargetParser. Mostly because the build of LLVMTargetParser now requires less dependencies (in my specific case, from 188 to 165).

Of course this does not reflect on a full compiler build...

  % cmake ../llvm-project/llvm -G Ninja -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_ASSERTIONS=ON -DLLVM_TARGETS_TO_BUILD="AArch64;ARM;X86"
  % time ninja -j1 LLVMTargetParser
  [165/165] Linking CXX static library lib/libLLVMTargetParser.a
  ninja -j1 LLVMTargetParser  44.86s user 4.55s system 98% cpu 50.027 total



  % cmake ../llvm-project/llvm -G Ninja -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_ASSERTIONS=ON -DLLVM_TARGETS_TO_BUILD="AArch64;ARM;X86;RISCV"
  % time ninja -j1 LLVMTargetParser
  [188/188] Linking CXX static library lib/libLLVMTargetParser.a
  ninja -j1 LLVMTargetParser  56.60s user 5.70s system 98% cpu 1:03.45 total


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D156710/new/

https://reviews.llvm.org/D156710