-
-
Notifications
You must be signed in to change notification settings - Fork 5.7k
Closed
Labels
Description
If I'm interpreting llvm/llvm-project#97975 (comment) correctly, LLVM's problematic excess precision handling might not be in use with bfloat. If we can verify this, maybe we can just get rid of the demote pass for BFloat16 and
have_bf16.The other condition is that all our targets that do not support bfloat (i.e., most of them) properly legalize
bfloatinstructions to something that they do support. IIRC I encountered some instruction selection failures when first working on this, but those might have been fixed now that we're on a more recent version of LLVM.
Originally posted by @maleadt in #55417 (comment)