JDevlieghere accepted this revision.
JDevlieghere added a comment.
This revision is now accepted and ready to land.

A possible alternative would be to export LLVM_LIT_ARGS from the llvm build so 
that lldb inherits it. I'm pretty much on the fence: on the one hand this 
doesn't really seems like something that should be exported, but on the other 
hand it'd be nice to not have to duplicate the option and having to update the 
default if it changes in llvm.

LGTM in the sense that I have no strong opinion.


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

https://reviews.llvm.org/D107700

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

Reply via email to