RFR: 8292275: javac does not emit SYNTHETIC and MANDATED flags for parameters by default [v8]

Hannes Greule duke at openjdk.org
Sat Apr 29 19:03:23 UTC 2023


> With this change, javac emits the MethodParameters attribute in cases where the JLS requires the information about synthetic and mandated parameters to be stored (see issue).
> Parameter names are *not* emitted unless the `-parameter` flag is set.
> 
> The relevant changes are in `ClassWriter`, where we go through the params to see if we need the attribute if the `-parameter` flag is not set (if it is set, both names and flags will be emitted).
> For records, the mandated flag wasn't set at all, this is solved by the one line fix in `JavacParser`.
> 
> The changes to `CreateSymbols` and `ClassReader` are needed as they weren't able to deal with missing names in the attribute.
> I also had to update some tests as they got a new constant pool entry.
> 
> Only the mandated flag is covered by tests at the moment, as the occurrences are well-specified in the JLS.
> Please let me know if you want tests for specific appearances of synthetic parameters.

Hannes Greule has updated the pull request incrementally with one additional commit since the last revision:

  Use ternary operator
  
  Co-authored-by: liach <7806504+liach at users.noreply.github.com>

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/9862/files
  - new: https://git.openjdk.org/jdk/pull/9862/files/5ac3a214..400e1ec9

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=9862&range=07
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=9862&range=06-07

  Stats: 4 lines in 1 file changed: 0 ins; 2 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/9862.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/9862/head:pull/9862

PR: https://git.openjdk.org/jdk/pull/9862


More information about the compiler-dev mailing list