[PATCH] D33650: MachineVerifier: Verify liveins list

Krzysztof Parzyszek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 30 11:20:44 PDT 2017


kparzysz added a comment.

I don't think we should be aborting on legal code.  Maybe we should add an option to treat all verifier reports as warnings (i.e. continue the compilation even if issues were reported)?

Also, didn't you want to eventually get rid of the concept of pristine registers?  Doing that would require the opposite---having them as live-ins almost everywhere.  Is this patch meant as a short-term solution?


Repository:
  rL LLVM

https://reviews.llvm.org/D33650





More information about the llvm-commits mailing list