[PATCH] D78203: [VP,Integer,#2] ExpandVectorPredication pass

[Simon Moll via Phabricator via llvm-commits]

simoll planned changes to this revision.
simoll marked 3 inline comments as done.
simoll added a comment.

In D78203#2355876 <https://reviews.llvm.org/D78203#2355876>, @frasercrmck wrote:

> Aside from my nitpicks, I was wondering if it's possible to add tests for the other strategies. Even if that involved a strategy that you could force on the command-line, since I know we don't have real TTIs just now. It would be nice to see that the pass behaves in the various ways it's meant to.

I like that idea! I'll add some cl::opts to override the target strategy for tests. Thx :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78203