Integrated: 8294014: Remove redundant UseCompiler conditions
Julian Waters
jwaters at openjdk.org
Tue Sep 20 12:26:47 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).
This pull request has now been integrated.
Changeset: 0f28cb06
Author: Julian Waters <jwaters at openjdk.org>
Committer: Tobias Hartmann <thartmann at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/0f28cb06ab9de649dedbe93f5d4e30fb779532d9
Stats: 3 lines in 2 files changed: 0 ins; 0 del; 3 mod
8294014: Remove redundant UseCompiler conditions
Reviewed-by: kvn, thartmann
-------------
PR: https://git.openjdk.org/jdk/pull/10342
More information about the hotspot-compiler-dev
mailing list