RFR: 8267800: Remove the '_dirty' set in BCEscapeAnalyzer
Dean Long
dlong at openjdk.java.net
Wed May 26 20:20:04 UTC 2021
On Wed, 26 May 2021 14:29:35 GMT, Denghui Dong <ddong at openjdk.org> wrote:
> Hi,
>
> Could I have a review of this change?
>
> The content of `_dirty` in `BCEscapeAnalyzer` is only updated when processing `_aaload`. And it will not affect the results of the analysis because its content is never used.
> IIUC, I think we should remove this set.
>
> Thanks,
> Denghui
I did a little source code archeology, and _dirty was used when bcEscapeAnalyzer.cpp was added by https://bugs.openjdk.java.net/browse/JDK-6339956. But the changes made by https://bugs.openjdk.java.net/browse/JDK-6488063 caused _dirty to be unused.
-------------
PR: https://git.openjdk.java.net/jdk/pull/4208
More information about the hotspot-compiler-dev
mailing list