[PATCH] D129715: [LoongArch] Heuristically load FP immediates by movgr2fr from materialized integer

Gong LingQin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 14 20:11:17 PDT 2022


gonglingqin added inline comments.


================
Comment at: llvm/lib/Target/LoongArch/LoongArchISelLowering.cpp:874
+    return true;
+  // f64 imm is legal if the bitcasted integer can be materialized within 2
+  // instructions.
----------------
xen0n wrote:
> nit: `bitcast` -- the verb "cast"'s past participle is itself, so is the compound word "bitcast".
Thanks. I will change that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129715



More information about the llvm-commits mailing list