[PATCH] D63973: [MachineVerifier] Improve checks of target instructions operands.

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 26 03:21:10 PST 2019


lebedev.ri added a comment.

In D63973#1759970 <https://reviews.llvm.org/D63973#1759970>, @jonpa wrote:

> In D63973#1754038 <https://reviews.llvm.org/D63973#1754038>, @kparzysz wrote:
>
> > Fixed the Hexagon failures in 824b25fc02dc4544adae55e6451d355f4c6d7055 <https://reviews.llvm.org/rG824b25fc02dc4544adae55e6451d355f4c6d7055>.
>
>
> Thank you!
>
> SystemZ test case fixed by 3ec193f <https://reviews.llvm.org/rG3ec193fb527e697faac4ef8f30934dd7bce849a7>. This means that the only remaining failing test case is SPARC/fp128.ll, which fails with expensive checks enabled, and I have XFAIL:ed it.


I think we should get this in as-is, the existing failure can be addressed later and we won't risk having new failure creep in.


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

https://reviews.llvm.org/D63973





More information about the llvm-commits mailing list