[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