[PATCH] D102755: [AArch64] Add cost tests for bitreverse

[Irina Dobrescu via Phabricator via llvm-commits]

Rin marked an inline comment as done.
Rin added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:299
+      return LegalisationCost.first * Entry->Cost + 1;
+    if (TLI->getValueType(DL, RetTy, true) != MVT::Other)
+      return LegalisationCost.first * Entry->Cost;
----------------
dmgreen wrote:
> Should this be `if (Entry) return LegalisationCost.first * Entry->Cost;` now?
Yeah, you're right, I have changed it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102755