RFR: 8273408: java.lang.AssertionError: typeSig ERROR on generated class property of record [v3]
Guoxiong Li
gli at openjdk.java.net
Thu Sep 16 12:24:28 UTC 2021
> Hi all,
>
> After processing annotation, if new source files are generated, the compiler should re-parse and re-enter the source files so that the type and symbol can keep right. The compiler always holds this convention in the past. But when the feature `record` is added, the convention is broken. In this bug, the compiler doesn't catch or enter the new/good type for record component after processing annotation. So some issues may occur at the later phases.
>
> This patch fixes the type of the record component so that the phases `Gen` and `ClassWriter` can get the right type. And a test case is added.
>
> Thanks for taking the time to review.
>
> Best Regards,
> -- Guoxiong
Guoxiong Li has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:
- Add test cases about unknown type and annotation.
- Fix comments.
- Merge branch 'master' into JDK-8273408
- Treat source as comment mistakenly. Remove the comment symbol.
- Polish
- 8273408: java.lang.AssertionError: typeSig ERROR on generated class property of record
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/5511/files
- new: https://git.openjdk.java.net/jdk/pull/5511/files/7b756883..f262a85c
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=5511&range=02
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=5511&range=01-02
Stats: 904 lines in 89 files changed: 653 ins; 85 del; 166 mod
Patch: https://git.openjdk.java.net/jdk/pull/5511.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/5511/head:pull/5511
PR: https://git.openjdk.java.net/jdk/pull/5511
More information about the compiler-dev
mailing list