[PATCH] D53237: [RISCV] Implement RV64D codegen

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 14 13:07:03 PST 2019


efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: test/CodeGen/RISCV/double-fcmp.ll:211
+; RV64IFD-NEXT:    and a0, a0, a1
+; RV64IFD-NEXT:    seqz a0, a0
+; RV64IFD-NEXT:    xori a0, a0, 1
----------------
The seqz here shouldn't be necessary, I think.  But that doesn't need to be fixed in this patch.


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

https://reviews.llvm.org/D53237





More information about the llvm-commits mailing list