[clang] [llvm] [NVPTX] Add 'activemask' builtin and intrinsic support (PR #79768)

Justin Lebar via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 29 10:04:07 PST 2024


jlebar wrote:

> I was planning on updating this to use the new instrinsic for the newer version. Alternatively we could make __activemask the builtin which expands to both versions, but I'm somewhat averse since we should target the instruction directly I feel.

Yes, I agree that the builtin shouldn't have a "polyfill".  At least, the LLVM builtin should not have a polyfill -- I guess I'm neutral on whether the clang builtin does.

You can change clang in this same patch if you want, but if you want to do it separately, that's also fine by me.  I'll approve this one.  I think that covers all my outstanding review requests from you?  LMK if I missed any.

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


More information about the cfe-commits mailing list