It seems the relative address in AArch64 disassembly is dec, not hex.
Details
Diff Detail
- Repository
- rLNT LNT
Event Timeline
lnt/server/ui/static/lnt_profile.js | ||
---|---|---|
140–144 | 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. |
lnt/server/ui/static/lnt_profile.js | ||
---|---|---|
140–144 | Thank you for review, please check these changes ) |
I think the base commit is not in the repo that runs the CI. See what happened to me before I run git fetch:
% git show 6a3f7bbaa221aa13acb72947363acbc95a1bf583:lnt/server/ui/static/lnt_profile.js fatal: Path 'lnt/server/ui/static/lnt_profile.js' exists on disk, but not in '6a3f7bbaa221aa13acb72947363acbc95a1bf583'
Hm... this patch is on top of 6ba06570364b936c6bfb631e9d2e4f1374446bad ( https://github.com/llvm/llvm-lnt/commit/6ba06570364b936c6bfb631e9d2e4f1374446bad )
And file lnt/server/ui/static/lnt_profile.js is presented there for sure
Maybe I did something wrong when created this revision and now this patch cannot be applied properly
Hmm 6ba06570364b936c6bfb631e9d2e4f1374446bad is a very odd commit... why did that go in without any reviews..
This is weird, your base commit is on https://reviews.llvm.org/rLNT6a3f7bbaa221aa13acb72947363acbc95a1bf583, but 6ba06570364b936c6bfb631e9d2e4f1374446bad was identical to this original change.
Ah that's probably why it fails. I've reverted 6ba06570364b936c6bfb631e9d2e4f1374446bad. Did you push 6ba06570364b936c6bfb631e9d2e4f1374446bad by mistake @slydiman ? I'll revert it and land this change
I think it is no need to revert 6ba06570364b936c6bfb631e9d2e4f1374446bad - it looks like right solution
I've moved my changes to https://reviews.llvm.org/D110174
No it's fine , we're good now. I don't like 6ba06570364b936c6bfb631e9d2e4f1374446bad because it assumes consistencies between disassemblers which is not the case.
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.