[PATCH] D113199: [AArch64] Combine fptoi.sat(fmul) to fixed point cvtf

[Dave Green via Phabricator via llvm-commits]

dmgreen added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64InstrInfo.td:3770
+
+  def : Pat<(i32 (to_int_sat (fmul f16:$Rn, fixedpoint_f16_i32:$scale), i32)),
+            (!cast<Instruction>(INST # SWHri) $Rn, $scale)>;
----------------
SjoerdMeijer wrote:
> Do we need to predicate this rule and the next on `HasFullFP16`?
Hmm. We haven't for any of the others here above or below. I don't think the nodes will be legal if we don't have fullfp16 - so they will never be possible to match against. I can add them if you think that's cleaner.


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

https://reviews.llvm.org/D113199