[PATCH] D71637: [PatternMatch] Add support for matching ConstantExpr.

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

It looks like we've always matched ConstantExprs for binary operators, but not for other operations.  It probably makes sense to be more consistent.

If we're going to do this, probably makes sense to change OneOps_match/TwoOps_match at the same time.

Can you add testcases for a few of the existing transforms that are changing?  There are too many transforms for it to make sense to cover everything, but it makes sense to illustrate what's actually changing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71637