mib added inline comments.
================
Comment at: lldb/examples/python/crashlog.py:552-557
+ self.symbol_data[image_uuid]["symbols"].append({
+ "name": json_frame['symbol'],
+ "type": "code",
+ "size": 0,
+ "address": pc,
+ })
----------------
Same problem here I guess: IIUC if `pc = image['base'] + frame_offset`, then I
believe `frame_offset` is not the address of the beginning of the function, but
rather an address in the middle of the function (either at a callsite, or at
the crash site).
================
Comment at: lldb/examples/python/crashlog.py:669
+ r'(0x[0-9a-fA-F]{4,}) +' # addr (4
chars or more)
+ r'((.*)(?:(?: +\+ +)([0-9]+))|[^\s]+)' # symbol +
offset
)
----------------
@kastiglione may be you have a better idea how to handle `symbol + offset`
where ` + offset` might be optional.
================
Comment at: lldb/examples/python/crashlog.py:880
+ "size": 0,
+ "address": frame_addr,
+ })
----------------
I'm not sure if I should subtract the offset from the `frame_addr` so the
symbol is at the right address.
Currently, the symbol name is `symbol + offset`, so I didn't have to do
anything subtraction.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146765/new/
https://reviews.llvm.org/D146765
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
