Compared to RV64F, supporting RV64D is very straight-forward. This patch introduces the necessary instruction patterns.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
Update to fix a typo in a comment and reflect changes in double-intrinsics.ll (which now exhaustively checks double intrinsics).
Comment Actions
LGTM
test/CodeGen/RISCV/double-fcmp.ll | ||
---|---|---|
211 | The seqz here shouldn't be necessary, I think. But that doesn't need to be fixed in this patch. |
Comment Actions
Rebased on the latest RV64F update. As @efriedma pointed out, fptosi/fptoui produce poison if the input won't fit in the target type. Just like the equivalent case in RV64F, fcvt.l[u].d can typically be used rather than fcvt.w[u].d because of this.
Comment Actions
Patch refresh. No functional changes since last time. Just awaiting an LGTM on the dependent RV64F patch.
The seqz here shouldn't be necessary, I think. But that doesn't need to be fixed in this patch.