[PATCH] D69653: [GlobalISel] Match table opt: fix a bug in matching num of operands

Roman Tereshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 30 22:20:08 PDT 2019


rtereshin added a comment.

In D69653#1728175 <https://reviews.llvm.org/D69653#1728175>, @arsenm wrote:

> Testcase?


Doesn't exist upstream. Match tables gain about one extra num operands' check (for `G_INTRINSIC_W_SIDE_EFFECTS`), but it doesn't affect the behavior as far as I can tell, as in those cases the number of operands semantically is in fact stable, just not evident from static properties of the generic opcode itself.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69653





More information about the llvm-commits mailing list