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

lgtm with a comment



================
Comment at: test/Driver/cl-options.c:525
 
+// RUN: %clang_cl -### /c -flto %s 2>&1 | FileCheck -check-prefix=LTO %s
+// LTO: -flto
----------------
This needs `--` before `%s`, otherwise if `%s` expands to e.g. `/Users/foo` it 
will be interpreted as the `/U` option :-)

Same thing below.


https://reviews.llvm.org/D30239



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

Reply via email to