labath added a subscriber: zturner.
labath added inline comments.

================
Comment at: lldb/lit/Register/x86-mm-xmm-read.test:1
+# REQUIRES: x86
+# RUN: %clang %p/Inputs/x86-mm-xmm-read.s -o %t
----------------
mgorny wrote:
> labath wrote:
> > Does this mean "the host is an x86 system", or "x86 is a configured llvm 
> > target"? My impression is that in means the latter 
> > <https://github.com/llvm-mirror/lldb/blob/master/lit/lit.cfg.py#L49>, but 
> > we obviously want the former here.
> Do you have any idea how to achieve the former? I can't find anything looking 
> like it. Though I suppose it's entirely possible that there's no such thing 
> atm.
Yeah, it looks like there isn't anything like that at the moment. Maybe you 
could add something like `system-x86` and use that in this test ?

+ @zturner, if he has any ideas here.


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

https://reviews.llvm.org/D60325



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

Reply via email to