asb added a comment.
Herald added a subscriber: edward-jones.

This is looking great, the only remaining code comment I have is that 
getExtensionFeatures needs a comment describing it.

The remaining issue I have is more of a spec issue - do canonical ordering 
requirements apply to extension categories? e.g. is sfoo_xbar illegal because 
it should be ordered xbar_sfoo? From my reading of the note below table 22.1 in 
the spec I think it is illegal and ordering of non-standard user-level vs 
standard supervisor-level vs nons-standard supervisor-level should be checked. 
Do you agree with my interpretation?


https://reviews.llvm.org/D45284



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

Reply via email to