EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.

This LGTM, but I have a question: Should we add these dylib definitions right 
away so that it's easier to adopt the ABI change in future? This will allow 
legacy dylibs to support this ABI change in future. If you agree we can make 
that change as a follow up commit.


https://reviews.llvm.org/D27387



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

Reply via email to