[PATCH] D41870: [CodeGen] Don't print "pred:" and "opt:" in -debug output
Matthias Braun via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 9 09:26:23 PST 2018
MatzeB accepted this revision.
MatzeB added a comment.
This revision is now accepted and ready to land.
I think this change make sense given that the `pred:` and `opt:` information is not actually part of the MachineOperand but of MCInstrDesc.
I agree that having the extra "pred:" and "opt:" would make understanding easier in debugging situations but I would consider that secondary and do it in a later/a separate commit. When we do this we should enhance the .mir parser so it can skip over these prefixes so that printing them stays legal syntax.
Anyway for now I think this patch is fine.
https://reviews.llvm.org/D41870
More information about the llvm-commits
mailing list