[PATCH] D32744: [globalisel] Improve legalizer DEBUG_ONLY output.

Daniel Sanders via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 9 10:42:05 PDT 2017


dsanders added a comment.

I believe PrintTo will be valid since it's occurs after the instruction we're working on. If we're folding multiple instructions into one we'll be following the operands backwards, away from PrintTo.

For PrintFrom: At the moment, we only erase the current instruction and we leave operands we fold into it for the trivially-dead check in case there's another user of one of the defs for that operand. If that will continue to be the case, then PrintFrom is safe.


https://reviews.llvm.org/D32744





More information about the llvm-commits mailing list