[PATCH] D131253: [AArch64] Add bf16 select handling

[Dave Green via Phabricator via llvm-commits]

dmgreen added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:8440
+
+  // If we are lowering a f16 and we do not have fullf16, convert to a f32 in
+  // order to use FCSELSrrr
----------------
tschuett wrote:
> Did you say *b*f16 in the title?
Yep it is explained in the summary. The short version is that we are adding this for bf16, but can use the same method for f16 so save a few instructions. Or put another way we are adding lowering for f16 and re-using it for bf16.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131253/new/

https://reviews.llvm.org/D131253