On Sat, Jun 16, 2018 at 3:26 PM, Eliot Miranda <eliot.miranda at gmail.com>
wrote:

> Hi Chris,
>
> On Wed, Jun 13, 2018 at 8:34 PM, Chris Cunningham <cunningham.cb at gmail.com
> > wrote:
>
>> Hi.
>>
>> I was running the full test suite (which takes a LONG time), and it
>> encountered a #halt in Decompiler>>pushTemporaryVariable:
>>
>> When trying to figure out where this was, it appears to be in
>> DecompilerTests>>testDecompilerInClassesCNtoCZ.
>>
>> Appears to be because the image is now borked - unusuable.  Any attempt
>> to browse one of those classes or methods throws an error about the change
>> file failing (primGetPosition: failed).
>>
>> Is this related to the "don't log changes while tests are running"?  Or
>> something else.
>>
>
> As stated earlier it is due to bugs in the decompiler/debugger
> (specifically block start to temp name mapping) in the full block regime.
> You should find this fixed in  Compiler-eem.383/Compiler-eem.384.  Thank
> you _very much_ for spotting this.  It would have been an awful black mark
> on the new release.
>
Thank you for fixing it.  Just verifying that it truly does work.
And with a recent/decent cog/spur VM, the full decompiler test of all
methods in the image really isn't that slow, either (fast, even - 45
seconds on my machine).
Thanks,
Chris

> [snip]
> _,,,^..^,,,_
> best, Eliot
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20180714/8dbc505c/attachment.html>