rmaprath added a comment. In http://reviews.llvm.org/D14293#281183, @rengolin wrote:
> In http://reviews.llvm.org/D14293#281176, @jroelofs wrote: > > > I think this ^ is a reasonable solution. > > > I'm fine with that, as long as everyone's happy. > > 160 XFAILs are ok (as long as you're fixing them), disabling the tests makes > no sense. :) > > cheers, > --renato Great! I will update the patch http://reviews.llvm.org/D14292 to include the X-FAILS and then update the current patch to remove the hack which prevents the tests being run. Thanks. - Asiri http://reviews.llvm.org/D14293 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
