Jay Foad
Jay Foad
Thanks for following up on this!
> `__builtin.canonicalize` in common code This patch is about the LLVM intrinsic, not the Clang builtin, so the title should say `llvm.canonicalize`
> Compiler messages on HIP SDK for Windows Please rewrite this to say what the patch does or what problem it fixes.
> This change is in preparation for being able to disable intrinsics for targets that are not enabled. Why is it required for that? Is there some requirement that LLVM...
> 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...
Note that `llvm-project/llvm/.gitignore` contains: ``` runtimes/* !runtimes/*.* ``` I have seen some related issues like #2498 and #1050 but this one seems to be slightly different.
I can no longer reproduce it.