zturner added a comment.

I'll still wait and see what Pavel and/or Tamas say, but if we are going to go 
this route, I would rather the argument be called `cflags` and now `env_flags`. 
 Xfailing a test based on an arbitrary environment variable just seems like 
something we shouldn't be doing.  I'd rather prime the environment however we 
need in order to get the test to run, than disable it and lose test coverage.


http://reviews.llvm.org/D18481



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

Reply via email to