[PATCH] D13035: LiveIntervalAnalysis: Avoid multiple connected liveness components
Quentin Colombet via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 21 17:28:31 PDT 2015
qcolombet added a comment.
Hi Matthias,
Nice catch!
Right now, this is on the users of shrinkUses to fix the multi component splitting.
The rationale, I think :), is that we want the users to be aware of the any virtual registers that would be created. This is not the case in that patch.
The bottom line is I believe shrinkToUses should return true in those situations and we should fix the users of shrinkToUses, if need to.
Indeed, I, for instance, guess that the register coalescer would already do the right thing if we return true, since it creates new virtual registers, and wouldn’t have to fix it.
What do you think?
Cheers,
-Quentin
Repository:
rL LLVM
http://reviews.llvm.org/D13035
More information about the llvm-commits
mailing list