Pierre-vh wrote:

> > Sorry, I should have clearly mentioned that. Yes, it is for my followup 
> > change #80908. In #80908, we changed the type of LLVM builtin but kept the 
> > corresponding clang builtin unchanged to avoid breaking existing uses.
> 
> Don't see how that could be related; you can losslessly bitconvert between 
> i16 and bfloat

I guess it's an oversight from that function to not allow FP <-> INT casts when 
they have the same width?
It seems like it'd let `<4 x i16>` to  `<4 x bf16>` pass.

I think the right fix is teaching that function about FP <-> INT casts. 
What do you think @nikic ?



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

Reply via email to