refactor(nativemem): rename CODECACHE category to NATIVE_SYMBOLS

The "CodeCache" name is async-profiler's, and collides with the JVM's
JIT code cache. This category measures something unrelated: the
profiler's own per-native-library symbol tables (used to symbolicate
native frames), not JVM-managed code. Rename the NativeMem category and
its JFR label to native_symbols so the metric is unambiguous.

The existing CODECACHE_NATIVE_SIZE_BYTES counter keeps its name for
continuity; only the new NativeMem category is renamed.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: rkennke

ddprof-lib/src/main/cpp/nativeMem.h

@@ -31,7 +31,7 @@
   X(CALLTRACE, "calltrace")                                                    \
   X(DICTIONARY, "dictionary")                                                  \
   X(THREAD_FILTER, "thread_filter")                                            \
-  X(CODECACHE, "codecache")                                                    \
+  X(NATIVE_SYMBOLS, "native_symbols")                                          \
   X(LINE_TABLES, "line_tables")                                                \
   X(PERF, "perf")                                                              \
   X(THREAD_LOCAL, "thread_local")                                              \

ddprof-lib/src/main/cpp/profiler.cpp

@@ -1696,9 +1696,10 @@ Error Profiler::dump(const char *path, const int length) {
     Counters::set(CODECACHE_NATIVE_SIZE_BYTES, native_libs.memoryUsage());
     Counters::set(CODECACHE_RUNTIME_STUBS_SIZE_BYTES,
                   native_libs.memoryUsage());
-    // CodeCache size is a recomputed gauge (not alloc/free deltas), so mirror it
-    // as an absolute. It shares the accuracy caveats of the counter above.
-    NativeMem::setLive(NM_CODECACHE, native_libs.memoryUsage());
+    // CodeCache here is the profiler's native-symbol tables (not the JVM code
+    // cache). Its size is a recomputed gauge (not alloc/free deltas), so mirror
+    // it as an absolute. It shares the accuracy caveats of the counter above.
+    NativeMem::setLive(NM_NATIVE_SYMBOLS, native_libs.memoryUsage());
 
     Error err = Error::OK;
     // rotateDictsAndRun rotates the dictionaries, takes lockAll() around the

ddprof-lib/src/test/cpp/nativeMem_ut.cpp

@@ -94,12 +94,12 @@ TEST_F(NativeMemTest, NegativeLiveClampedInSample) {
 // setLive() overwrites the live value (gauge semantics) but still advances the
 // peak, which is retained when the gauge shrinks.
 TEST_F(NativeMemTest, SetLiveOverwritesAndAdvancesMax) {
-    NativeMem::setLive(NM_CODECACHE, 4096);
-    EXPECT_EQ(4096, NativeMem::live(NM_CODECACHE));
-    EXPECT_EQ(4096, NativeMem::max(NM_CODECACHE));
-    NativeMem::setLive(NM_CODECACHE, 1024);  // gauge shrinks
-    EXPECT_EQ(1024, NativeMem::live(NM_CODECACHE));
-    EXPECT_EQ(4096, NativeMem::max(NM_CODECACHE));  // peak retained
+    NativeMem::setLive(NM_NATIVE_SYMBOLS, 4096);
+    EXPECT_EQ(4096, NativeMem::live(NM_NATIVE_SYMBOLS));
+    EXPECT_EQ(4096, NativeMem::max(NM_NATIVE_SYMBOLS));
+    NativeMem::setLive(NM_NATIVE_SYMBOLS, 1024);  // gauge shrinks
+    EXPECT_EQ(1024, NativeMem::live(NM_NATIVE_SYMBOLS));
+    EXPECT_EQ(4096, NativeMem::max(NM_NATIVE_SYMBOLS));  // peak retained
 }
 
 // Every category exposes a distinct, non-empty name.