[PATCH] D45157: [RegUsageInfoCollector] Don't assume the alias of a defined reg is always already in the set.

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 3 15:48:56 PDT 2018


qcolombet accepted this revision.
qcolombet added a comment.
This revision is now accepted and ready to land.

You're right, re-reading the comment indeed, not all the aliases are here (nor should they be!).



================
Comment at: test/CodeGen/SystemZ/ipra.ll:15
+define void @fn1() {
+  br label %1
+
----------------
Nit: Please use `opt -instnamer` to get rid of the implicit variables


https://reviews.llvm.org/D45157





More information about the llvm-commits mailing list