thakis added a comment.

This approach looks good to me.

Is there any chance we could have a test for this? See `test/Driver` for 
examples; use `python bin\llvm-lit.py 
../llvm/tools/clang/test/Driver/my_test.cpp` to run a single test, `ninja 
check-clang` to run all of them. You need some unix binaries to run tests -- if 
you don't have those flying around, you can download gnuwin-7.zip from 
http://commondatastorage.googleapis.com/chromium-browser-clang/index.html?path=tools/
 and add them to your %PATH%.


https://reviews.llvm.org/D36860



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

Reply via email to