labath accepted this revision.
labath added a comment.

This looks much better, though I'd just change one tiny thing: in the macro 
"canonicalization" step, I'd define a custom macro, instead of reusing one of 
the system ones (`_M_ARM` and friends), as those are reserved identifiers and 
it's bad for your karma to define those.

It might also be nice to make the linking of the relevant plugins conditional 
in the cmake files, but judging by a quick search, there isn't a very straight 
forward way to do that (CMAKE_SYSTEM_PROCESSOR would be variable to look at, 
but it's values aren't very standardized).


Repository:
  rLLDB LLDB

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

https://reviews.llvm.org/D61543



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

Reply via email to