RFR: 8310076: Reduce inclusion of bytecodeStream.hpp

Ioi Lam iklam at openjdk.org
Fri Jun 16 15:29:07 UTC 2023


On Fri, 16 Jun 2023 13:10:57 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

> Looks good but why would generateOopMap.hpp be a popular header file?

It's included via frame_x86.inline.hpp -> oopMapCache.hpp -> generateOopMap.hpp.

None of these includes seem necessary. I'll try to fix that in a separate issue. See [JDK-8310225](https://bugs.openjdk.org/browse/JDK-8310225)

Thanks @coleenp and @dholmes-ora for the review.

Passed tier 1, tier 2, and builds-tier5

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

PR Comment: https://git.openjdk.org/jdk/pull/14481#issuecomment-1594867322
PR Comment: https://git.openjdk.org/jdk/pull/14481#issuecomment-1594869623


More information about the hotspot-dev mailing list