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