PBQP bugfix

Lang Hames lhames at gmail.com
Mon Feb 2 22:18:48 PST 2015


Hi Jonas,

Apologies for the delay - I've committed my fix for this in r227942.

I haven't had a chance to try it out on your ARM test case yet, but hope to
soon. Please let me know if it doesn't fix the problem for your out-of-tree
target.

Cheers,
Lang.


On Fri, Jan 30, 2015 at 11:10 AM, Lang Hames <lhames at gmail.com> wrote:

> Hi Jonas,
>
> Thanks very much for tracking this down!
>
> I think the best approach to fixing this is to have the PBQP register
> allocator recognize these nodes early and never represent them in the
> graph. Any matrix that has only one column must be connected to a node that
> has only one solution, and this should be allocated up-front to simplify
> the solver's job.
>
> I'll have a fix for this later today.
>
> Cheers,
> Lang.
>
> On Thu, Jan 29, 2015 at 9:25 AM, Jonas Paulsson <
> jonas.paulsson at ericsson.com> wrote:
>
>>  Hi,
>>
>>
>>
>> This is a bugfix for the PBQP register allocator. The case for a
>> one-column matrix must be handled in the MatrixMetadata constructor.
>>
>>
>>
>> See commit message for further explanation,
>>
>>
>>
>> Jonas Paulsson
>>
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150202/793b1cea/attachment.html>


More information about the llvm-commits mailing list