[PATCH] D109148: [DAGCombiner][VP] Fold zero-length or false-masked VP ops

[Simon Moll via Phabricator via llvm-commits]

simoll added a comment.

LGTM. We should cleanup the `VPIntrinsics.def` file some time .. originally every property was attached to //some// entity (intrinsic and/or sdnode) by placing the property macro in the scope of the entity. Now, we are seeing the ID being repeated in the property macros, making it independent of the scope. Let's discuss this on the call. This should not hold back the patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109148