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

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 23 08:37:16 PDT 2021


fhahn added inline comments.


================
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
----------------
lebedev.ri wrote:
> 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.
Yes exactly. I did just that in 7fb6d9f9588f, thanks!


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