[PATCH] D90462: [VE][TTI] don't advertise vregs/vops

Kazushi Marukawa via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 4 03:24:38 PST 2020


kaz7 added a comment.

@fhahn -san,  thank you for letting us know about `llvm/test/Analysis/CostModel/`.  I didn't know those regression tests.  We, @simoll and me actually, will add several cost-model tests in near future.  However, the purpose of this patch is disabling vector regs advertisement and it is tested well by newly added regression tests, in my opinion.  Therefore, I think this is LGTM.  Please let us know if you have any suggestions about it.   And thank you very much for reviewing this patch!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90462



More information about the llvm-commits mailing list