Proposed DCE change related to dynamic field access

HaxeFoundation / haxe

Haxe - The Cross-Platform Toolkit

https://haxe.org

6.17k stars 654 forks source link

Proposed DCE change related to dynamic field access #6996

Open Simn opened 6 years ago

Simn commented 6 years ago

Issues like #6994 keep coming up, so I propose the following change to DCE:

If we have any dynamic field access (d : Dynamic).fieldName, we keep fieldName on all classes that we're keeping anyway.

ncannasse commented 6 years ago

I agree, but maybe only in DCE=std ? I would still like a DCE=full to be really full. We can have an additional dce level "all" that applies to all classes but still does this Dynamic bookchecking

back2dos commented 6 years ago

I'd like some way of getting full dce and debugging the thing to just get rid of the dynamic access altogether.