[PATCH] D100823: [RISCV] Implement the pseudo compare builtin.
Craig Topper via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 20 16:53:27 PDT 2021
craig.topper added a comment.
Thinking about this more, I'd like to consider doing this in the backend specifically for the compares for a few reasons.
-vmsgt(u).vv intrinsic already selects vmslt(u).vv with swapped operands in the backend. This was needed for SEW=64 on RV32.
-Inconsistent that some compare IR intrinsics accept vector or scalar second argument, but some only support scalar.
-We can probably generate better code for masked vmsge(u).vx if we defer expansion to post register allocation.
-Unifies the compare intrinsic implementation in the frontend.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100823/new/
https://reviews.llvm.org/D100823
More information about the cfe-commits
mailing list