[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