erichkeane wrote:

> > However, I'd suggest instead adding the clspv attribute in a separate 
> > review/commit. It isn't really related to this one, other than this has a 
> > slight dependency on it.
> 
> I could add that in a separate pr; that means that one will have to be merged 
> before this one because we have to replace that one use of `assume` in the 
> clcfunc.h header with the new attribute either before or at the same time as 
> we remove the `assume` spelling.

Yep, understood.  I'll do my best to do quick review cycles on the other patch.

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

Reply via email to