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

[Wei Mi via llvm-commits]

No problem. Thanks again for help doing the measurement.

Wei.

On Thu, Mar 10, 2016 at 5:09 PM, Quentin Colombet <qcolombet at apple.com> wrote:
> 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
>>>
>