RFR: 8337563: NMT: rename MEMFLAGS to MemTag [v10]
Gerard Ziemski
gziemski at openjdk.org
Thu Sep 12 15:40:35 UTC 2024
> Please review this cleanup, where we rename `MEMFLAGS` to `MemTag`.
>
> `MEMFLAGS` implies that we can use more than one at the same time, but those are exclusive values, so `MemTag` is a more suitable name.
>
> This fix also includes a cleanup of all the related function/template parameter names and local variable names.
>
> Testing is pending...
>
> Note: there is more history in old closed PRs [https://github.com/openjdk/jdk/pull/20497](https://github.com/openjdk/jdk/pull/20497) and [https://github.com/openjdk/jdk/pull/20472](https://github.com/openjdk/jdk/pull/20472)
Gerard Ziemski has updated the pull request incrementally with two additional commits since the last revision:
- Afshin's feedback, record_virtual_memory_type -> record_virtual_memory_tag
- Afshin's feedback
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/20872/files
- new: https://git.openjdk.org/jdk/pull/20872/files/8e576cf1..1765579b
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=20872&range=09
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=20872&range=08-09
Stats: 42 lines in 24 files changed: 0 ins; 0 del; 42 mod
Patch: https://git.openjdk.org/jdk/pull/20872.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/20872/head:pull/20872
PR: https://git.openjdk.org/jdk/pull/20872
More information about the serviceability-dev
mailing list