[clang] [Clang][CodeGen] Loose the cast check when emitting builtins (PR #81669)

Pierre van Houtryve via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 14 04:15:56 PST 2024


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


More information about the cfe-commits mailing list