[PATCH] D15302: [Greedy regalloc] Replace analyzeSiblingValues with something new [Part1]

[Quentin Colombet via llvm-commits]

Hi Wei,

Apologizes, neither Michael or I had time to do the measurements yet.

I hope we can look into it by the end of next week.

I’ll keep you posted.

Cheers,
-Quentin


> On Mar 10, 2016, at 4:00 PM, Wei Mi <wmi at google.com> wrote:
> 
> Ping.
> 
> On Fri, Jan 29, 2016 at 10:48 AM, Wei Mi <wmi at google.com> wrote:
>> wmi updated this revision to Diff 46399.
>> wmi added a comment.
>> 
>>> Hi Mikhail,
>> 
>>> Could you check how this patch impacts our performance?
>> 
>> 
>> Thanks. This is the patch "hoistspill + split-mode-speed [Part1] + redo rematerialize [Part2] + remove analyzeSiblingValues [Part3]" merged together, which I used to do the performance testing.
>> // I divided the patches for easier review. To merge the separate patches together, it needs to resolve some conflicts.
>> 
>> 
>> Repository:
>>  rL LLVM
>> 
>> http://reviews.llvm.org/D15302
>> 
>> Files:
>>  include/llvm/CodeGen/LiveRangeEdit.h
>>  lib/CodeGen/InlineSpiller.cpp
>>  lib/CodeGen/LiveRangeEdit.cpp
>>  lib/CodeGen/RegAllocBase.cpp
>>  lib/CodeGen/RegAllocBase.h
>>  lib/CodeGen/RegAllocBasic.cpp
>>  lib/CodeGen/RegAllocGreedy.cpp
>>  lib/CodeGen/RegAllocPBQP.cpp
>>  lib/CodeGen/RegisterCoalescer.cpp
>>  lib/CodeGen/Spiller.h
>>  lib/CodeGen/SplitKit.cpp
>>  lib/CodeGen/SplitKit.h
>>  test/CodeGen/AArch64/aarch64-deferred-spilling.ll
>>  test/CodeGen/ARM/subreg-remat.ll
>>  test/CodeGen/SPARC/spill.ll
>>  test/CodeGen/X86/avx512-bugfix-25270.ll
>>  test/CodeGen/X86/fold-push.ll
>>  test/CodeGen/X86/hoist-spill.ll
>>  test/CodeGen/X86/new-remat.ll
>>  test/CodeGen/X86/ragreedy-hoist-spill.ll
>>  test/CodeGen/X86/vselect-minmax.ll
>>