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

[Quentin Colombet via llvm-commits]

qcolombet added a comment.

Hi Wei,

I think we will need to wait for Tom to double check what happened for AMDGPU.

One question though, this revision ended up being the combination of the 3 parts, right?

Cheers,
-Quentin


================
Comment at: test/CodeGen/X86/hoist-spill.ll:2
@@ +1,3 @@
+; RUN: llc < %s | FileCheck %s
+
+; grep 'Spill' |sed 's%.*\(-[0-9]\+(\%rsp)\).*%\1%g' |sort |uniq -d |awk '{if (/rsp/); exit -1}'
----------------
You could check where the spills actually are. But it already looks pretty good now :).


Repository:
  rL LLVM

http://reviews.llvm.org/D15302