[PATCH] D128899: [LoongArch] Add codegen support for fpround, fpextend and converting between signed integer and floating-point

Sat Jul 2 00:47:06 PDT 2022

xen0n added a comment.

Still not quite sure about the necessity of the `_LA64` nodes/pats (is it again because of tblgen intricacies?) but the rest looks okay.

================
Comment at: llvm/lib/Target/LoongArch/LoongArchISelLowering.h:42
+
+  FTINT_LA,
+
----------------
I'm not quite sure about what the "LA" suffix means. Aren't we obviously LoongArch? Or is this meaning "L(64-bit integer), Any FP precision"? This seems likely until I find both "w" and "l" `ftintrz`'s are pattern-matched from this.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128899