[Lldb-commits] [PATCH] D12556: Use eAddressClassCode for address lookup for opcodes
Greg Clayton via lldb-commits
lldb-commits at lists.llvm.org
Thu Sep 3 11:48:05 PDT 2015
clayborg requested changes to this revision.
clayborg added a comment.
This revision now requires changes to proceed.
Did you check who is all calling this? Is it only places that know that an address is code? It seems like we have might have different clients expecting different things out of this function call. For the stack backtracing code, we want to force the address to be code. Others might want to know if it is data. One idea is to add a bool parameter like "bool address_is_always_code". If this is true we call:
code_addr = target->GetOpcodeLoadAddress (code_addr, address_is_always_code ? eAddressClassCode : GetAddressClass());
More information about the lldb-commits