[PATCH] D131378: [LoongArch] Add codegen support for bitreverse

[Lu Weining via Phabricator via llvm-commits]

SixWeining added inline comments.


================
Comment at: llvm/lib/Target/LoongArch/LoongArchISelLowering.cpp:77
+  setOperationAction(ISD::BITREVERSE, MVT::i8, Custom);
+  setOperationAction(ISD::BITREVERSE, MVT::i16, Custom);
+  if (Subtarget.is64Bit()) {
----------------
SixWeining wrote:
> How about leave it to legal and use bitrev.4b+srli.w/d. I'm not sure which is fast and better.
Sorry, I mean bitrev.w/d+srli.w/d.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131378