[llvm] [LLVM] Add `LLVMIntrinsicCopyName` to LLVM C API (PR #112197)

Rahul Joshi via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 4 07:53:38 PST 2024


jurahul wrote:

> I haven't been able to follow the overall discussion about intrinsic names. I think the code change is fine in terms of quality, but whether it should go ahead depends on where the overall discussion is going.

My latest post summarizes the status of trimming target specific intrinsics here: https://discourse.llvm.org/t/rfc-compress-intrinsic-name-table/82412/49

Its either illegal or has high logistical cost. And since code size savings seems to be not significant from that change, this one (whose motivation was an even smaller data size savings) seems even less significant to pursue. So I am planning to close this and other reviews once the thread settles down. I have minor turly NFC improvement, which I'll implement sometime soon.

https://github.com/llvm/llvm-project/pull/112197


More information about the llvm-commits mailing list