[PATCH] D110141: Fixed relative address in LNT profile control-flow graph.

Pavel Kosov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 21 04:36:26 PDT 2021


kpdev42 added inline comments.


================
Comment at: lnt/server/ui/static/lnt_profile.js:141
       if (addressString.startsWith('#'))
-          return addressCurrent + parseInt(addressString.substring(1), 16);
+          return addressCurrent + parseInt(addressString.substring(1), 10);
       else
----------------
tnfchris wrote:
> Thanks for the patch! How about checking for the `0x` to determine whether it's dec or hex.  This would be more robust. and do the same for the absolute address part below.
Thank you for review, please check these changes )


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

https://reviews.llvm.org/D110141



More information about the llvm-commits mailing list