[PATCH] D147434: [LV] Improve register pressure estimation if MaxLocalUsers is zero

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 4 09:39:07 PDT 2023


fhahn added a comment.

Do you have any performance data motivating the change and ruling out any regressions?



================
Comment at: llvm/test/Transforms/LoopVectorize/X86/interleave-count.ll:38
+loop:
+  %p = phi double * [%dst, %entry], [%p.next, %loop]
+  store double -1.000000e+00, ptr %p, align 8
----------------
Please update the test to use opaque pointers. Also, it would be good to put up a patch to just add the test separately and then only include the changes caused by the patch in the diff


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147434



More information about the llvm-commits mailing list