labath added a comment.

Would you say that "pc" is a reasonable value for the "vendor" field for the 
win+aarch64 combo? I am asking because I don't have a clue about that, and 
given that this platform is being brought up right now, changing this now would 
be way easier than doing it later. (The reason why things don't work is the 
incompatibility between the two things that compute the ArchSpec, but that can 
also be fixed by changing the other mechanism, if that is better/more correct.) 
My guess is the other mechanism is ArchSpec::SetArchitecture function, line 
928...


Repository:
  rLLDB LLDB

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68939/new/

https://reviews.llvm.org/D68939



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

Reply via email to