[PATCH] D68267: [MBB LiveIn lists, MachineVerifier, SystemZ] New method isLiveOut() and mverifier improvement.

Jonas Paulsson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 7 10:47:18 PST 2019


jonpa added a comment.

In D68267#1737195 <https://reviews.llvm.org/D68267#1737195>, @RKSimon wrote:

> @jonpa Any movement on this? The buildbots are still red - maybe time to revert?


I was hoping that someone with knowledge of the X86 would fix the tests now that I pointed out the causes. It would be a shame to revert an improvement to the machine verifier if it now discovers broken tests with expensive checks enabled, I think...

I have made an improvement for the SystemZ backend (https://reviews.llvm.org/D67437) which has already been reviewed which depends on this patch, so it would be really nice to keep this. Would it be possible to temporarily XFAIL those tests only with expensive checks enabled?


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

https://reviews.llvm.org/D68267





More information about the llvm-commits mailing list