dwblaikie wrote:

Hmm, do other builtins actually end up as symbol names? I think most of them 
lower to an instruction or two - I guess this one doesn't lower to an actual 
symbol, only a DWARF symbol - but maybe that's still different enough it should 
use an llvm or clang in the name... (sorry, I think I saw this was discussed 
somewhere on the review, but I can't find where now... :/)

I guess it came up here: 
https://github.com/llvm/llvm-project/pull/79230#discussion_r1591335258 and at 
the time it still had `llvm` in it, wasn't clear conversation that showed how 
the alternative conclusion was reached.

https://github.com/llvm/llvm-project/pull/79230
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to