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

Irina Dobrescu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 9 02:31:45 PDT 2021


Rin added a comment.

In D102755#2807464 <https://reviews.llvm.org/D102755#2807464>, @CarolineConcatto wrote:

> Hey @Rin,
> I don't think you need bitreverse-cost-model.ll, you can use only bitreverse.ll. 
> Why some costs are not being changed in bitreverse.ll?

Hi Carol, yeah, you're right, I'll remove it. And I'm working on making the costs adequate as well, which will require adjustments in bitreverse.ll


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102755



More information about the llvm-commits mailing list