[PATCH] D124383: llvm-reduce: Do not try to delete frame instructions

Markus Lavin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 25 07:37:27 PDT 2022


markus added inline comments.


================
Comment at: llvm/test/tools/llvm-reduce/mir/remove-frame-destroy.mir:5
+
+# CHECK-INTERESTINGNESS: S_NOP 0
+
----------------
Is this criteria for interestingness safe given that we have two `S_NOP 0` instructions in the block? Could it be that the first one gets eliminated and then it is still considered interesting but the final RESULT-NEXT fails because there is now a `S_NOP 0` in the middle?


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

https://reviews.llvm.org/D124383



More information about the llvm-commits mailing list