[PATCH] D98633: [LV] Move exact FP math check out of Requirements.

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 15 09:09:09 PDT 2021


lebedev.ri added a comment.

Not familiar enough with the code, but it totally makes sense to me, thanks!



================
Comment at: llvm/test/Transforms/LoopVectorize/PowerPC/reg-usage.ll:224-237
 ;CHECK-LABEL: fp128_
 ;CHECK: LV(REG): VF = 1
 ;CHECK: LV(REG): Found max usage: 2 item
 ;CHECK: LV(REG): RegisterClass: PPC::GPRRC, 2 registers
 ;CHECK: LV(REG): RegisterClass: PPC::VRRC, 2 registers
 entry:
   br label %for.body
----------------
I'm guessing the test needs to change so that the LV cost model is still constructed and printed?
I think you can just commit this as NFC.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98633



More information about the llvm-commits mailing list