[PATCH] D19293: [profile] LLVM support for memory-mapping profile counters

Vedant Kumar via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 19 18:53:21 PDT 2016


vsk added inline comments.

================
Comment at: lib/Transforms/Instrumentation/InstrProfiling.cpp:390
@@ -385,1 +389,3 @@
 
+void InstrProfiling::emitCounterPadding() {
+  auto PageSize = llvm::sys::Process::getPageSize();
----------------
Hm, I think I misunderstood your question.

To clarify, I'm not sure whether or not other linkers have the same behavior, so I thought it would be safer to always emit the padding.


http://reviews.llvm.org/D19293





More information about the llvm-commits mailing list