I updated the code based on your suggestions with one difference. In 
Target::MergeArchitecture I always do a merge if the current architecture and 
the specified architecture is compatible because if the architecture was fully 
specified then the MergeFrom is a no op and the check you suggested (checking 
for OS and Vendor) isn't sufficient because in some case (e.g. on android) we 
still have to update the environment part of the triple.


http://reviews.llvm.org/D8057

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/



_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits

Reply via email to