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

[Roger Ferrer Ibanez via Phabricator via llvm-commits]

rogfer01 added a comment.

I understand this is a legacy pass manager. Would it make sense to adapt it to the New PM as well?



================
Comment at: llvm/test/CodeGen/Generic/expand-vp.ll:1
+; Partial expansion cases (still VP with parameter expansions).
+; RUN: opt --expandvp --expandvp-override-evl-transform=Legal --expandvp-override-mask-transform=Legal -S < %s | FileCheck %s --check-prefix=LEGAL_LEGAL
----------------
simoll wrote:
> rogfer01 wrote:
> > Just a suggestion: `llvm/utils/update_test_checks.py` can reduce the cost of maintaining this test (I understand we'll have to extend it when VP FP lands).
> How about we switch to an auto-generated test once caching is in and the code has stabilized?
> Without caching the output is very verbose.. also auto-generated tests are really sentinels for change not actually //tests//
Yep. Agreed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78203