[PATCH] D13035: LiveIntervalAnalysis: Avoid multiple connected liveness components

[Quentin Colombet via llvm-commits]

qcolombet added a comment.

I guess if it is needed we could have an additional optional parameter (like SmallVectorImpl<unsigned> *) to return the virtual registers created.
Seems like it is not needed for now.

The thing that I do not like is that it seems to me that the handling of multi component is not homogeneous between the non-subregister and the subregister case.

> computeDeadValues() is also used during the initial Liveness computation in which the return value is not checked (because we cannot have deadPhi values at that place).


How hard would it be to account for the returned value?

Thanks,
-Quentin


Repository:
  rL LLVM

http://reviews.llvm.org/D13035