[llvm] [LLVM] Change Intrinsic::ID to encode target and intrinsic index (PR #113576)
Jay Foad via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 5 05:50:50 PST 2024
jayfoad wrote:
> Have you checked whether this does help incremental builds in practice?
I checked and it does seem to help. Adding a new AMDGPU intrinsic did not cause any other target's `.cpp` files to be rebuilt. (But TBH I'm still not a huge fan of this patch, just because it seems like extra complexity that we don't really need.)
https://github.com/llvm/llvm-project/pull/113576
More information about the llvm-commits
mailing list