bkramer accepted this revision.
bkramer added a comment.

This looks good, thanks for the patch. When this goes in please watch the 
windows buildbots as plugin support on windows is somewhat broken and we may 
have to disable tests there.

Also, while this is a strict improvement and should go in I'm not so sure that 
it solves the "plugins in libclang" issue. I couldn't get it to work even with 
this patch due to the way we build libclang in Release mode.


http://reviews.llvm.org/D15729



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

Reply via email to