[PATCH] D118612: [NFC] Move FoldingSetNodeIDRef::ComputeHash and FoldingSetNodeID::ComputeHash definitions to header

Thu Feb 3 03:45:26 PST 2022

yurai007 added a comment.

In D118612#3290697 <https://reviews.llvm.org/D118612#3290697>, @aeubanks wrote:

> if you want the most performance, shouldn't you be building with ThinLTO `-DLLVM_ENABLE_LTO=Thin`? or is this helping even with LTO?

We discussed LTO topic a bit here: https://reviews.llvm.org/D118385#3278901 On my local build I can use LTO but in general not all distros enable it for Clang/LLVM packages. I'm not sure whether change would give any gains on top of LTO build.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118612