[PATCH] D31188: [AntiDepBreaker] Use liveins as well in StartBlock

[Matthias Braun via Phabricator via llvm-commits]

MatzeB added a comment.

In https://reviews.llvm.org/D31188#714712, @echristo wrote:

> Committed thusly:
>
> echristo at athyra ~/s/llvm> git svn dcommit
>  Committing to https://llvm.org/svn/llvm-project/llvm/trunk ...
>
>   	M	lib/AsmParser/LLParser.cpp
>   	M	lib/CodeGen/AggressiveAntiDepBreaker.cpp
>   	M	lib/CodeGen/CriticalAntiDepBreaker.cpp
>
> Committed r299124
>
> and... here:
>
> echristo at athyra ~/s/llvm> git svn dcommit
>  Committing to https://llvm.org/svn/llvm-project/llvm/trunk ...
>
>   	A	test/CodeGen/PowerPC/pristine-and-livein.mir
>
> Committed r299125
>
> *sigh*


FWIW: I am pretty sure "getPristineRegs is not accurately considering shrink wrapping puts registers not saved in certain blocks." as mentioned in the commit message is not true! getPristineRegs() doesn't have to do that!

I also don't understand the fix here: Adding live-in registers when the code here appears to be searching for live-out registers seem like an odd fix to me (I can see how it helps though as it will just mark more registers as occupied).


https://reviews.llvm.org/D31188