Vitja Makarov, 27.01.2011 08:25:
> Ok. I've moved it into ParseTreeTransforms and created branch:
>
> https://github.com/vitek/cython/commit/a8e957ec29f0448ee7c43bd3969012772d09b236
Hmm, currently, it runs after type analysis:
_check_c_declarations,
AnalyseExpressionsTransform(self),
RemoveUnreachableCode(self),
This /may/ make a difference, considering that there may be declarations in
the dead code that may have an impact on the remaining code.
However, I would prefer running the dead code removal as early as possible,
even before analysing the declarations. If users rely on dead code making
live code work, I'd consider that a bug. That would also simplify the
removal operation, as the tree tends to be a lot simpler in earlier stages
(e.g. before optimisations, WithTransform, etc.).
Stefan
_______________________________________________
Cython-dev mailing list
Cython-dev@codespeak.net
http://codespeak.net/mailman/listinfo/cython-dev
