Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deconstruction: Properties on the left may not do proper flow analysis #13827

Open
jcouv opened this issue Sep 15, 2016 · 1 comment
Open

Deconstruction: Properties on the left may not do proper flow analysis #13827

jcouv opened this issue Sep 15, 2016 · 1 comment

Comments

@jcouv
Copy link
Member

jcouv commented Sep 15, 2016

See comment

Method PreciseAbstractFlowPass.VisitDeconstructionAssignmentOperator does not have special handling for properties, unlike VisitCompoundAssignmentOperator.
It's worth noting that VisitCompoundAssignmentOperator seems to lack test coverage though (see issue #12160)

Split this issue from the umbrella deconstruction work items #11299.

@gafter
Copy link
Member

gafter commented Sep 19, 2016

This is an artifact of the phase ordering error described in #13270.

@jcouv jcouv changed the title Deconstruction: Properties on the left Deconstruction: Properties on the left may not do proper flow analysis Sep 21, 2016
@VSadov VSadov modified the milestones: 2.0 (RTM), 2.0 (RC) Oct 6, 2016
@jcouv jcouv modified the milestones: 2.1, 2.0 (RTM) Jan 11, 2017
@jcouv jcouv modified the milestones: 2.1, 2.2 Mar 3, 2017
@jcouv jcouv modified the milestones: 15.2, 15.3 Mar 11, 2017
@jcouv jcouv modified the milestones: 15.later, 15.3 Apr 5, 2017
@jaredpar jaredpar added the Bug label Jul 7, 2017
@jcouv jcouv modified the milestones: 15.6, 16.0 Nov 2, 2017
@jcouv jcouv modified the milestones: 16.0, 16.1, 16.2 Apr 18, 2019
@jcouv jcouv removed this from the 16.2 milestone Jun 19, 2019
@jcouv jcouv added this to the Compiler.Next milestone Jun 19, 2019
@jaredpar jaredpar modified the milestones: Compiler.Next, Backlog Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants