[PATCH] D97033: [RISCV] Use custom isel for vector indexed load/store intrinsics.

Zakk Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 19 00:41:04 PST 2021


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

LGTM!!

I'm little curious, what's benefit to have two GenericTables for load/store? 
Why not add a `isLoad` field in FilterClass and using one GenericTable for VLX and VSX?
because we have two identical FilterClass, RISCVVLX and RISCVVSX.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97033



More information about the llvm-commits mailing list