Re: [eigen] Allocation policy of eigen decompositions

[ Thread Index | Date Index | More lists.tuxfamily.org/eigen Archives ]


2010/4/20 Benoit Jacob <jacob.benoit.1@xxxxxxxxx>:
> 2010/4/20 Benoit Jacob <jacob.benoit.1@xxxxxxxxx>:
>> 2010/4/20 Adolfo Rodríguez Tsouroukdissian <adolfo.rodriguez@xxxxxxxxxxxxxxxx>:
>>>
>>>
>>> 2010/4/20 Adolfo Rodríguez Tsouroukdissian <dofo79@xxxxxxxxx>
>>>>
>>>>
>>>> On Mon, Apr 19, 2010 at 7:36 PM, Benoit Jacob <jacob.benoit.1@xxxxxxxxx>
>>>> wrote:
>>>>>
>>>>> Sorry, my bad! I added this eval() for debugging and forgot to remove it.
>>>>>
>>>>> Can you retry now? This should remove 1 heap allocation. I really
>>>>> don't know where the 3 others can be coming from, but that would be a
>>>>> bug that we (I...) introduced, so you don't have to block your stuff
>>>>> because of it.
>>>>
>>>> Removing eval() did the trick. I was seeing four allocations because the
>>>> call to applyHouseholderOnTheLeft is inside a loop :s, so problem solved.
>>>>
>>>
>>> FYI: A secondary effect of removing the above eval() is that these tests are
>>> now failing:
>>>
>>> householder_5
>>> householder_6
>>> householder_7
>>
>> I can reproduce the householder_6 failure, not the others. Thanks for
>> letting me know.
>
> I can now confirm the 5,6,7 failures with vectorization disabled.
>
> I am afraid that I introduced a huge bug that so far had been hidden
> by this eval(). Investigating...

This is now fixed, can you retry?

Benoit



Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/