Skip stack accesses unless requested, as the memory profiler runtime
does not currently look at or report accesses for these addresses.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/test/Instrumentation/HeapProfiler/stack.ll | ||
---|---|---|
5 | I see. Just skimmed the legacy PM deprecation email and saw note about this. Might be worth a wider announcement? |
llvm/lib/Transforms/Instrumentation/MemProfiler.cpp | ||
---|---|---|
129 | Perhaps call this memprof-instrument-stack, the usage will be clear from the name itself? Also I think moving this to L110 is better since the comment at L111 seems to indicate all flags following it are debug flags. |
Perhaps call this memprof-instrument-stack, the usage will be clear from the name itself?
Also I think moving this to L110 is better since the comment at L111 seems to indicate all flags following it are debug flags.