[PATCH] D124663: LNT: Only use text segment when isSO

Thomas Preud'homme via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 6 02:54:30 PDT 2022


thopre accepted this revision.
thopre added a comment.
This revision is now accepted and ready to land.

LGTM with the suggested codestyle changes



================
Comment at: lnt/testing/profile/cPerf.cpp:536-538
+        /* Adjust the symbol to a value relative to the start of the load address
+            to match up with registerNewMapping.  */
+        M->Adjust -= segmentStart;
----------------
LLVM code style does not use tab. Can you change those 3 lines to use whitespaces instead?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124663



More information about the llvm-commits mailing list