[PATCH] D92538: [RISCV] Merge FMV_H_X_RV32/FMV_H_X_RV64 into a single opcode. Same with FMV_X_ANYEXTH_RV32/RV64

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 3 10:59:25 PST 2020


craig.topper added a comment.

In D92538#2431700 <https://reviews.llvm.org/D92538#2431700>, @jrtc27 wrote:

> Ah, I hadn't realised the F comment got lost in 741b04b0b7912611a8a5b7e74462e87b8930a116 <https://reviews.llvm.org/rG741b04b0b7912611a8a5b7e74462e87b8930a116>, I was looking at our fork which is currently still based on 11. This looks good to me now.

I'm not sure why I dropped it unless I was basing it on the lack of comment for the RV64 patterns.


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

https://reviews.llvm.org/D92538



More information about the llvm-commits mailing list