[clang] [llvm] [clang][HLSL] Add WaveIsFirstLane() intrinsic (PR #103299)
Farzon Lotfi via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 19 11:47:27 PDT 2024
farzonl wrote:
> > We have this work tracked here: #99158
> > there should be some dxil specific tasks.
>
> Seems like most boxes would be checked by this PR, except Sema checks:
>
> * what kind of Sema checks would be required for this one?
> Also, the intrinsic name in the issue is using camel case vs snake case for this PR. But seems like existing ones like thread_id are using snake case (same thing on the SPIR-V backend). So shouldn't we remain consistent?
good point, the only thing I could think to add would be `TheCall->setType(BoolTy);` But I'm not sure if its worth it.
https://github.com/llvm/llvm-project/pull/103299
More information about the cfe-commits
mailing list