apoch / epoch-language

Home of the Epoch Programming Language Project
Other
72 stars 3 forks source link

Function lexical scope metadata is aliased #110

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Function::TypeInference in Function.cpp currently generates two copies of the 
lexical scope metadata for a function: one under the function's given name 
(mangled for overloading) and one under the automatically generated scope name. 
This is excessive and wasteful and should be improved; metadata for the 
automatic scope should not be necessary since the lexical scope should always 
be located via looking up the mangled function name.

Original issue reported on code.google.com by don.ap...@gmail.com on 15 Feb 2012 at 7:33

GoogleCodeExporter commented 9 years ago
Aliasing has been removed and nothing broke, so apparently some time along the 
way this got fixed and I forgot about it.

Original comment by don.ap...@gmail.com on 14 Nov 2012 at 3:52

GoogleCodeExporter commented 9 years ago
So compiling ERA broke with this change in place. Will have to take a deeper 
look.

Original comment by don.ap...@gmail.com on 17 Nov 2012 at 6:19

GoogleCodeExporter commented 9 years ago
OK, fixed for reals nao.

Original comment by don.ap...@gmail.com on 19 Nov 2012 at 4:46