pharo icon indicating copy to clipboard operation
pharo copied to clipboard

Symbolic bytecode improvement

Open hogoww opened this issue 1 year ago • 0 comments

As suggested #11263 , improvement of the printing of symbolic bytecodes with variables names for temps, remote temps, & instances variables

hogoww avatar Jul 29 '22 14:07 hogoww

Can you check because none of the builds is building and with the same error not related to git

Ducasse avatar Aug 12 '22 20:08 Ducasse

Absolutely no idea. I'll try to redo it sometime next week.

hogoww avatar Aug 12 '22 21:08 hogoww

NewUndeclaredWarning: CompiledCodeTest>>method (test is Undeclared) NewUndeclaredWarning: CompiledCodeTest>>method (test is Undeclared) [ERROR] 2022-08-12 22:57:29.000 growOldSpaceByAtLeast (Replacement/generated/64/vm/src/gcc3x-cointerp.c:48922):Required space is bigger than the headroom. Could not allocate

this error is strange.

Ducasse avatar Aug 13 '22 13:08 Ducasse

There is a consistent error in all builds:

Syntax Error on line 3: 'End of statement expected'

===================================================

1: UndefinedMethod arg1

2: 

3: 	^ arg1 + arg2

So this seems related...

guillep avatar Aug 16 '22 09:08 guillep

There is a consistent error in all builds:

Syntax Error on line 3: 'End of statement expected'

===================================================

1: UndefinedMethod arg1

2: 

3: 	^ arg1 + arg2

So this seems related...

Clearly, but I haven't touched any code using a + That's why I want to redo it, I'm hopping it is it might be a side effect that randomly occurred.

hogoww avatar Aug 16 '22 09:08 hogoww

Looking at the code, I wonder if this has something to do with decompilation.

It looks like this is a method with two args that are not properly decompiled. But the symbolic bytecode should have nothing to do with that... Unless the symbolic bytecode is used during decompilation...

guillep avatar Aug 16 '22 09:08 guillep