[PATCH] D73480: [VectorCombine] new IR transform pass for partial vector ops

[Sanjay Patel via Phabricator via llvm-commits]

spatel marked an inline comment as done.
spatel added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/VectorCombine.cpp:34
+STATISTIC(NumVecCmp, "Number of vector compares formed");
+DEBUG_COUNTER(VecCombineCounter, "vector-combine-transform",
+              "Controls transformations in vector-combine pass");
----------------
craig.topper wrote:
> spatel wrote:
> > craig.topper wrote:
> > > Is this counter used?
> > Oops - no. Copy/pasted from existing code. Looks like other passes do something like:
> >     if (!DebugCounter::shouldExecute(VecCombineCounter))
> >       continue;
> > 
> > I've never used that myself, so I could either add that code or remove the counter. Let me know if there's a preference. Not sure how we test it?
> Probably fine to drop it. I added the one to instcombine and I've only used it a couple times.
Removed:
rGa2a0f9a43a71


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73480