<div dir="ltr">Yes, I quite liked the things I've read about the PBQP allocator.<div><br></div><div>Given what the hardware folks have to go through to get 1% improvements in scalar code,</div><div>spending 20% (or whatever) compile time (under control of a flag) seems like nothing.</div><div>And falling back on "average code" is a little disingenuous.</div><div>People looking for performance don't care about average code;</div><div>they care about their own personal code, their important loop, and wonder</div><div>why there are unnecessary copies right <i>there</i> and wasn't this problem</div><div>solved ages ago?</div><div><br></div><div>Preston</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 11, 2018 at 11:00 AM, Quentin Colombet <span dir="ltr"><<a href="mailto:quentin.colombet@gmail.com" target="_blank">quentin.colombet@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le mar. 11 sept. 2018 à 10:23, Preston Briggs<br>
<<a href="mailto:preston.briggs@gmail.com">preston.briggs@gmail.com</a>> a écrit :<br>
<span class="">><br>
> Hi,<br>
><br>
> Using Chaitin's approach, removing a copy via coalescing could expose more opportunities for coalescing.<br>
> So he would iteratively rebuild the interference graph and check for more opportunities.<br>
<br>
</span>I am not sure Chaitin's approach would lead to a better coalescing<br>
than what LLVM does.<br>
I believe most of the copies that we see in the final code comes from<br>
live-range splitting (i.e., to avoid spill), not coalescing, but I am<br>
nitpicking :).<br>
<span class=""><br>
><br>
> Chaitin was also careful to make sure that the source and destination of a copy didn't interfere unnecessarily (because of the copy alone);<br>
> that is, his approach to interference was very precise. The idea of computing interference from intersecting ranges seems less precise,<br>
> though I owe Matthias examples demonstrating these supposed weaknesses.<br>
><br>
> Finally, when spills are introduced, more opportunities for coalescing can arise.<br>
> By separating coalescing and allocation, it seems like you miss an opportunity.<br>
><br>
<br>
</span>LLVM still eliminates copies at coloring time. The available registers<br>
are biases toward the colors used by whatever copies connected to<br>
current live-range.<br>
<br>
Also, to be fair, LLVM provides a graph coloring regalloc with the<br>
PBQP allocator.<br>
<br>
That said, I agree that we trade compile time for code quality, but<br>
pragmatically, it is hard to justify spending say 20% time in compile<br>
time to achieve maybe 1% speed up on average (numbers completely mad<br>
up :)). If you're willing to pay the price, the PBQP may be an option.<br>
ARM made a talk showing how it performed<br>
(<a href="https://llvm.org/devmtg/2014-10/Slides/PBQP-update-and-in-the-wild.pdf" rel="noreferrer" target="_blank">https://llvm.org/devmtg/2014-<wbr>10/Slides/PBQP-update-and-in-<wbr>the-wild.pdf</a>).<br>
<br>
What I am saying is, I believe you're right but there isn't a big<br>
enough incentive for a company to invest in such project.<br>
<div class="HOEnZb"><div class="h5"><br>
> I admit that the extra time spent rebuilding the interference graph<br>
> and attempting to coalesce can add up, but I hate that we (apparently) give up on quality<br>
> results to maybe save some compile time. Our machines are really fast these days;<br>
> lets spend their speed on something useful!<br>
><br>
> Thanks for the pointer and explanation,<br>
> Preston<br>
><br>
><br>
> On Tue, Sep 11, 2018 at 9:40 AM, Quentin Colombet <<a href="mailto:quentin.colombet@gmail.com">quentin.colombet@gmail.com</a>> wrote:<br>
>><br>
>> Hi Preston,<br>
>><br>
>> To summarize what Matthias said, LLVM approach to regalloc is split in<br>
>> two main phases:<br>
>> 1. Aggressive coalescing based on the "interference graph" of the<br>
>> variable in SSA (we don't actually build the interference graph). At<br>
>> that stage, we don't care about the whether or not the resulting<br>
>> representation is colorable with K register.<br>
>> 2. Priority based register allocation with live-range splitting.<br>
>> Basically when we are about to spill, instead of doing that right<br>
>> away, we have heuristics to split the live-range and re-enqueue the<br>
>> results.<br>
>><br>
>> The most high-level description of LLVM's regalloc that I can think of<br>
>> is available here:<br>
>> <a href="https://llvm.org/devmtg/2011-11/Olesen_RegisterAllocation.pdf" rel="noreferrer" target="_blank">https://llvm.org/devmtg/2011-<wbr>11/Olesen_RegisterAllocation.<wbr>pdf</a><br>
>><br>
>> Cheers,<br>
>> -Quentin<br>
>> Le lun. 10 sept. 2018 à 17:28, Matthias Braun via llvm-dev<br>
>> <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> a écrit :<br>
>> ><br>
>> ><br>
>> ><br>
>> > On Sep 10, 2018, at 5:25 PM, Matthias Braun <<a href="mailto:mbraun@apple.com">mbraun@apple.com</a>> wrote:<br>
>> ><br>
>> ><br>
>> ><br>
>> > On Sep 10, 2018, at 5:11 PM, Preston Briggs <<a href="mailto:preston.briggs@gmail.com">preston.briggs@gmail.com</a>> wrote:<br>
>> ><br>
>> > The phi instruction is irrelevant; just the way I think about things.<br>
>> > The question is if the allocator believes that t0 and t2 interfere.<br>
>> ><br>
>> > Perhaps the coalescing example was too simple.<br>
>> > In the general case, we can't coalesce without a notion of interference.<br>
>> ><br>
>> > There is also logic in `LiveRange::overlaps()` that considers live ranges as not overlapping in some cases where they are related via copy instructions. This will also effect the interference during the main allocation algorithm.<br>
>> ><br>
>> > My worry is that looking at interference by ranges of instruction numbers<br>
>> > leads to inaccuracies when a range is introduced by a copy.<br>
>> ><br>
>> ><br>
>> > I can't think of a case where we wouldn't have the same accuracy as a graph coloring allocator.<br>
>> > If you can construct an example where we don't I'd love to hear about it.<br>
>> ><br>
>> > If you wonder about the liveness information, you can perform experiments like this (I'm using a NOOP instruction with an implicit use operand to produce some artificial uses).<br>
>> ><br>
>> > $ cat test.mir<br>
>> > name: somefunc<br>
>> > body: |<br>
>> >   bb.0:<br>
>> >     %0:gr32 = MOV32ri 42<br>
>> >     JB_1 %bb.2, undef implicit $eflags<br>
>> >     JMP_1 %bb.2<br>
>> ><br>
>> >   bb.1:<br>
>> >     %1:gr32 = MOV32ri 17<br>
>> >     JMP_1 %bb.3<br>
>> ><br>
>> >   bb.2:<br>
>> >     NOOP implicit %0<br>
>> >     %1 = COPY %0<br>
>> >     JMP_1 %bb.3<br>
>> ><br>
>> >   bb.3:<br>
>> >     NOOP implicit %1<br>
>> ><br>
>> ><br>
>> ><br>
>> > $ llc -run-pass=liveintervals -debug-only=regalloc test.mir<br>
>> > ********** INTERVALS **********<br>
>> > %0 [16r,64B:0)[112B,144r:0)  0@16r weight:0.000000e+00<br>
>> > %1 [80r,112B:1)[144r,176B:0)[<wbr>176B,192r:2)  0@144r 1@80r 2@176B-phi weight:0.000000e+00<br>
>> > RegMasks:<br>
>> > ********** MACHINEINSTRS **********<br>
>> > # Machine code for function somefunc: NoPHIs<br>
>> ><br>
>> > 0B bb.0:<br>
>> >  successors: %bb.2(0x80000000); %bb.2(100.00%)<br>
>> ><br>
>> > 16B  %0:gr32 = MOV32ri 42<br>
>> > 32B  JB_1 %bb.2, implicit undef $eflags<br>
>> > 48B  JMP_1 %bb.2<br>
>> ><br>
>> > 64B bb.1:<br>
>> >  successors: %bb.3(0x80000000); %bb.3(100.00%)<br>
>> ><br>
>> > 80B  %1:gr32 = MOV32ri 17<br>
>> > 96B  JMP_1 %bb.3<br>
>> ><br>
>> > 112B bb.2:<br>
>> > ; predecessors: %bb.0<br>
>> >  successors: %bb.3(0x80000000); %bb.3(100.00%)<br>
>> ><br>
>> > 128B  NOOP implicit %0:gr32<br>
>> > 144B  %1:gr32 = COPY %0:gr32<br>
>> > 160B  JMP_1 %bb.3<br>
>> ><br>
>> > 176B bb.3:<br>
>> > ; predecessors: %bb.1, %bb.2<br>
>> ><br>
>> > 192B  NOOP implicit %1:gr32<br>
>> ><br>
>> > # End machine code for function somefunc.<br>
>> ><br>
>> ><br>
>> > If you look at the "intervals" (the class is a misnomer since nowadays it contains a list of ranges...) in the beginning you see that %0 and %1 do not overlap anywhere.<br>
>> ><br>
>> > - Matthias<br>
>> ><br>
>> ><br>
>> > But perhaps I should focus on the links and, as you suggested,<br>
>> > the debugging info.<br>
>> ><br>
>> > Thanks,<br>
>> > Preston<br>
>> ><br>
>> ><br>
>> ><br>
>> ><br>
>> > On Mon, Sep 10, 2018 at 5:02 PM, Matthias Braun <<a href="mailto:mbraun@apple.com">mbraun@apple.com</a>> wrote:<br>
>> >><br>
>> >><br>
>> >><br>
>> >> On Sep 10, 2018, at 4:53 PM, Preston Briggs <<a href="mailto:preston.briggs@gmail.com">preston.briggs@gmail.com</a>> wrote:<br>
>> >><br>
>> >><br>
>> >> > The underlying liveness datastructure is a list of ranges where each vreg is alive<br>
>> >> > (ranges in terms of instructions numbered). I remember a couple of later linear scan<br>
>> >> > papers describing the same thing (Traub <a href="http://et.al" rel="noreferrer" target="_blank">et.al</a>. being the first if I remember correctly).<br>
>> >> > That should be as accurate as you can get in terms of liveness information.<br>
>> >><br>
>> >> It depends on the details.<br>
>> >> For example, given<br>
>> >><br>
>> >> t0 = mumble<br>
>> >><br>
>> >> if (something) {<br>
>> >>   t2 = 3<br>
>> >> }<br>
>> >> else {<br>
>> >>   t3 = t0 + 3<br>
>> >>   print t0<br>
>> >> }<br>
>> >> t4 = phi(t2, t3)<br>
>> >><br>
>> >><br>
>> >> it's clear that t2 and t0 shouldn't interfere,<br>
>> >> but some folks might say the ranges overlap.<br>
>> >><br>
>> >><br>
>> >> Similarly,<br>
>> >><br>
>> >> t6 = mumble<br>
>> >> t7 = t6<br>
>> >> t8 = t6 + 5<br>
>> >> t9 = t7 + 10<br>
>> >> print t8, t9<br>
>> >><br>
>> >><br>
>> >> Chaitin points out that t6 and t7 shouldn't interfere,<br>
>> >> even though the live ranges overlap.<br>
>> >><br>
>> >> - We go out of SSA form before allocating registers so you won't see phi instruction.<br>
>> >> - In the second case the copy coalesceing pass should have coalesced t6 and t7.<br>
>> >><br>
>> >> I would expect both cases to work as you expect.<br>
>> >><br>
>> >> - Matthias<br>
>> >><br>
>> >><br>
>> >><br>
>> >> Anyway, I'll look at the links.<br>
>> >><br>
>> >> Thanks,<br>
>> >> Preston<br>
>> >><br>
>> >><br>
>> >><br>
>> >><br>
>> >>><br>
>> >>><br>
>> >>> We have separate aggressive coalescing pass before allocation. The register allocator will later perform live range splitting inside the main allocation loop as it seems fit.<br>
>> >>><br>
>> >>><br>
>> >>> I ask these questions because we (guys I work with) see loops<br>
>> >>> where there's a little register juggling that seems unnecessary.<br>
>> >>><br>
>> >>> Well your best bet is to learn reading the output of `-mllvm -print-machineinstrs -mllvm -debug-only=regalloc` (which can take a while)...<br>
>> >>><br>
>> >>><br>
>> >>> Is there a paper that describes what y'all do?<br>
>> >>><br>
>> >>><br>
>> >>> I'm only aware of a blog post:<br>
>> >>> <a href="http://blog.llvm.org/2011/09/greedy-register-allocation-in-llvm-30.html" rel="noreferrer" target="_blank">http://blog.llvm.org/2011/09/<wbr>greedy-register-allocation-in-<wbr>llvm-30.html</a><br>
>> >>> and a dev conference talk in 2011:<br>
>> >>> <a href="https://llvm.org/devmtg/2011-11/" rel="noreferrer" target="_blank">https://llvm.org/devmtg/2011-<wbr>11/</a><br>
>> >>><br>
>> >>> - Matthias<br>
>> >>><br>
>> >>><br>
>> >>> Thanks,<br>
>> >>> Preston<br>
>> >>><br>
>> >>><br>
>> >>> On Mon, Sep 10, 2018 at 9:57 AM, Matthias Braun <<a href="mailto:mbraun@apple.com">mbraun@apple.com</a>> wrote:<br>
>> >>>><br>
>> >>>> I would not describe LLVMs register allocator as linear scan, it's closer to graph coloring than linear scan IMO (though doesn't really matcher either approach).<br>
>> >>>><br>
>> >>>> RegAllocGreedy assigns the registers in an order based on the priority value computed in enqueu() we are not just scanning from top to bottom of the program. We also perform actual interference checks we just don't happen to build up an explicit interference graph up front.<br>
>> >>>><br>
>> >>>> - Matthias<br>
>> >>>><br>
>> >>>> On Sep 10, 2018, at 9:49 AM, Preston Briggs via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br>
>> >>>><br>
>> >>>> Why have we ended up using linear-scan register allocation<br>
>> >>>> by default (instead of, e.g., coloring)?<br>
>> >>>><br>
>> >>>> Thanks,<br>
>> >>>> Preston<br>
>> >>>><br>
>> >>>><br>
>> >>>> ______________________________<wbr>_________________<br>
>> >>>> LLVM Developers mailing list<br>
>> >>>> <a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a><br>
>> >>>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
>> >>>><br>
>> >>>><br>
>> >>><br>
>> >>><br>
>> >><br>
>> >><br>
>> ><br>
>> ><br>
>> ><br>
>> > ______________________________<wbr>_________________<br>
>> > LLVM Developers mailing list<br>
>> > <a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a><br>
>> > <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
><br>
><br>
</div></div></blockquote></div><br></div>