compnerd added a comment.

I think that splitting this up in a series of patches would be much better.  
The first patch should be to do the entirely mechanical change of the 
visibility attribute.  It is a separate library and needs its own visibility 
attribute.  That would significantly slim down this subsequent change.  A 
follow-up change could fix the build, and a third change adds the test 
harness/support the tests.


https://reviews.llvm.org/D37182



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

Reply via email to