jhuber6 wrote:

> > > This needs a release note, else I think this is fine. LGTM.
> > 
> > 
> > I added one under the non-comprehensive changes. Should I put it somewhere 
> > more specific?
> 
> Nope, that was me mis-reading the Github interface. LGTM.

Nice, thanks for the quick review. I'm going to try making the generic bit 
functions (e.g. `__builtin_ctzg`) work with these vestors as well. A future 
patch I'll like to add a new queryable feature for boolean vectors so people 
can start depending on this expanded support. Right now we can only check 
`__has_feature(ext_vector_type)` which will return an error on booleans until 
clang 15 and isn't really useful until this patch. Likely we'll wand a builtin 
for masked loads / stores as well.

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

Reply via email to