[PATCH] D59189: [perf][DebugInfo] follow up for "add SectionedAddress to DebugInfo interfaces"

Alexey Lapshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 13 13:00:42 PDT 2019


avl added a comment.

This looks OK for me(taking into account the comment). Let final approve would be done by David, since He reviewed original patch. 
David, Is it OK for you ?



================
Comment at: lib/ExecutionEngine/PerfJITEvents/PerfJITEventListener.cpp:270
+    object::SectionedAddress Address;
+    Address.Address = *AddrOrErr;
+
----------------
above two lines are not necessary. 


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59189/new/

https://reviews.llvm.org/D59189





More information about the llvm-commits mailing list