RFR: 8294014: Remove redundant UseCompiler conditions

Vladimir Kozlov kvn at openjdk.org
Mon Sep 19 20:08:52 UTC 2022


On Mon, 19 Sep 2022 13:46:09 GMT, Julian Waters <jwaters at openjdk.org> wrote:

> Several areas that use CompileBroker::should_compile_new_jobs() also contain a UseCompiler check which isn't needed, since the aforementioned already contains a check for this flag. In some areas (Such as with compileBroker.cpp which has AlwaysCompileLoopMethods) this duplicated check is required, so we can mark should_compile_new_jobs() and the related get_compilation_activity_mode() as inline to make the obvious optimization more obvious to the compiler used in the build (This should be fine, since both are small methods).

Good.

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

Marked as reviewed by kvn (Reviewer).

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


More information about the hotspot-compiler-dev mailing list