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());


http://reviews.llvm.org/D12556



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to