[PATCH] D81461: GlobalISel: Fix double printing new instructions in legalizer

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 10 17:47:26 PDT 2020


arsenm added a comment.

In D81461#2086302 <https://reviews.llvm.org/D81461#2086302>, @aditya_nandakumar wrote:

> Is there any interest in me checking in the above unit test? I initially didn't check it in as I thought this was more of a side effect of how things are implemented and not really how the APIs should be used.


I think tests that show the expected behavior are useful even if you aren't supposed to use it


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

https://reviews.llvm.org/D81461





More information about the llvm-commits mailing list