RFR: 8282721: HotSpot Style Guide should allow considered use of C++ thread_local [v4]

David Holmes dholmes at openjdk.java.net
Fri Mar 11 06:35:31 UTC 2022


> Style guide changes to support JDK-8282469 (PR https://github.com/openjdk/jdk/pull/7719). We no longer prohibit use of C++ `thread_local`, but allow it when there is an essential, and considered, need.
> 
> This is a modification of the Style Guide, so rough consensus among the HotSpot Group members is required to make this change. Only Group members should vote for approval (via the github PR), though reasoned objections or comments from anyone will be considered. A decision on this proposal will not be made before Friday 18-Mar-2022 at 12h00 UTC.
> 
> Since we're piggybacking on github PRs here, please use the PR review process to approve (click on Review Changes > Approve), rather than sending a "vote: yes" email reply that would be normal for a CFV.

David Holmes has updated the pull request incrementally with one additional commit since the last revision:

  Additional tweaks requested by @kbarrett

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/7720/files
  - new: https://git.openjdk.java.net/jdk/pull/7720/files/c307466c..5e2913e8

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7720&range=03
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7720&range=02-03

  Stats: 4 lines in 2 files changed: 0 ins; 0 del; 4 mod
  Patch: https://git.openjdk.java.net/jdk/pull/7720.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/7720/head:pull/7720

PR: https://git.openjdk.java.net/jdk/pull/7720


More information about the hotspot-dev mailing list