EricWF added a comment.

I don't think the tricky part of this patch is actually implementing the ABI 
macros. The tricky part is defining how libc++ should use the macros.  Some 
questions I would like to see answered:

1. How long is a major and minor ABI version supported?
2. When is the major and minor ABI version bumped?
3. How will maintaining multiple ABI versions affect code complexity and 
maintainability?

I would want to see documentation accompany this patch that anwsers some of 
these questions. It could we written in reStructured text after 
http://reviews.llvm.org/D12129 lands.


Repository:
  rL LLVM

http://reviews.llvm.org/D11740



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

Reply via email to