tree f40283409b6a3922e91f5b126e04bfac21217bc9
parent 9d04f82befe1cbbe21b39e07ab9255d06cb37872
author Steven Perron <stevenperron@google.com> 1545243462 +0000
committer GitHub <noreply@github.com> 1545243462 +0000
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJcGotGCRBK7hj4Ov3rIwAAdHIIAGsf3CK/2p00/lQeswcI5/K/
 iGRt62ao+yktEIUL4Sg0JYV0XM7hKc0na6+UzlDAYCjRTCrIexi4sv/QO/+FnDKY
 Rcm0sXt02Rdi+4IUa2TtDSIoL61pAZyhOhVHVw+Ny+rIivx4Vy1iKoegTxYSjabI
 2xwzSQ8cRaI5qtdAg/T5UX14ev+UiUaahoMSkFxMTcvanMkaM1o2gLiEQsAk6Ryh
 o2poHeYKZU4HIk0P1hAZjJqGuTzVmtzxZmNIpQVQB1KgVfnXcY71fgHXlry8hlFY
 FsAchht6IYvKgz8S4lf+D39TqGm+agdQvfIRPFz0dfCCfSSFuwp4YKX9YRgyty4=
 =2YJM
 -----END PGP SIGNATURE-----
 

Fix missing OpPhi after merge return. (#2248)

The function `UpdatePhiNodes` was being called inconsistently.  In one
case, the cfg had already been updated to include the new edge, and in
another place the cfg was not updated.  This caused the function to
miss flagging a block as needing new phi nodes.  I picked that the cfg
should not be updated before making the call.  I documented it, and
change the call sites to match.

Fixes #2207.