> Before this fix only strict fields were readable in the prologue phase. The 
> proposed fix should allow any instance fields of identity classes to be 
> readable in the prologue phase. This implies changes in flow analysis as 
> before we were only tracking final and strict fields. There is also some 
> "cooperation" needed in the code to detect cases when reading a field is not 
> allowed in the prologue phase. For example some methods in Resolve don't have 
> all the needed information at the moment they are dealing with some ASTs and 
> part of the processing needs to be done in Attr
> 
> TIA
> 
> This PR is a remake of https://github.com/openjdk/valhalla/pull/1490

Vicente Romero has updated the pull request incrementally with one additional 
commit since the last revision:

  more simplifications and bug fixes

-------------

Changes:
  - all: https://git.openjdk.org/valhalla/pull/1523/files
  - new: https://git.openjdk.org/valhalla/pull/1523/files/31c7b8c2..a5f2947d

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=valhalla&pr=1523&range=08
 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=1523&range=07-08

  Stats: 100 lines in 4 files changed: 29 ins; 36 del; 35 mod
  Patch: https://git.openjdk.org/valhalla/pull/1523.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/1523/head:pull/1523

PR: https://git.openjdk.org/valhalla/pull/1523

Reply via email to