fineg74 wrote:

> > I updated the branch with variadics support. I commented out __gpu_constant 
> > definition to make the build compile and since the issue with the backend 
> > is not related to this PR I'd suggest to do it as a follow up fix.
> 
> Workaround shouldn't happen in `spirvintrin.h` if we must have it, do it in 
> the library. Also IDK how printf works in SPIR-V, you might need some empty 
> vprintf stub or maybe it just compiles as normal.

The issue Karol reported has nothing to do with printf, because he reported it 
yesterday when the fix wasn't in the branch. IT is something in the backend. I 
can apply some other fix to make it build without touching spirvintrin.h

https://github.com/llvm/llvm-project/pull/174675
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to