rnk added a comment. In https://reviews.llvm.org/D43621#1019724, @ruiu wrote:
> This change looks good to me, but I think we shouldn't check in an executable > binary file. Can you create `ld.foo.exe` in the test? Since you don't > actually run ld.foo.exe, creating that executable should be very easy. If I read the diff correctly, it is an empty file, so this should just work. https://reviews.llvm.org/D43621 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
