sas requested review of this revision.
sas added a comment.

I saw the `bool is_arm = ` pattern only in `ObjectFileMachO.cpp`. I'm not sure 
about the specifics for Darwin targets, but having an object file with some 
functions in ARM and some functions in Thumb is valid, so I think checking for 
each symbol is the right thing to do.
As for the `GetAddressClass` function, I have another diff that does exactly 
what you are talking about. I'll upload it when this one goes in.


https://reviews.llvm.org/D39315



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

Reply via email to