https://github.com/jayfoad commented:
This patch highlights a problem that I have run into before: _if_ we want to support a single opcode (in this case BUNDLE) that may or may not load or store depending on its operands, then the current definitions of `MachineInstr::mayLoad` and `mayStore` are not good enough, because they just check a static property of the opcode. If `mayLoad` and `mayStore` were more sophisticated then this check would have been handled already at L1527. Having said that, in the short term I have no objection to this patch. https://github.com/llvm/llvm-project/pull/166211 _______________________________________________ llvm-branch-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
