[llvm-commits] [llvm] r132814 - /llvm/trunk/lib/CodeGen/TailDuplication.cpp

[Rafael Avila de Espindola]

On 11-06-09 07:05 PM, Cameron Zwarich wrote:
> Was this motivated by a test case?

Yes, but it depends on us being more aggressive about early tail 
duplication. Right now that code path is disable for anything but 
indirectbrs.

I found the issue by changing shouldTailDuplicate to return true for 
small bbs in the PreRegAlloc case, and bootstrapping clang with 
-tail-dup-size=8.

I guess it should be possible to reproduce it now by replacing all 
braches with indirectbr in the test...

I have one more fix to commit, and then it should be possible to 
benchmark the different thresholds for both taildup passes without 
crashing :-)

> Cameron

Cheers,
Rafael