[PATCH] [modules] PR20507: Avoid silent textual inclusion.
Richard Smith
richard at metafoo.co.uk
Wed Jul 1 17:26:14 PDT 2015
I'm fine with special-case hacks to keep the current Darwin module maps working, so long as they're relatively self-contained and very unlikely to trigger in any other cases. (We do equally ugly things to parse broken libstdc++ headers.)
I agree that `exclude` makes more sense for `requires excluded` as a "supported" feature, but if this is another egregious hack for Darwin, I don't think it matters which we choose. (Maybe we should also check the module name here?)
http://reviews.llvm.org/D10423
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the cfe-commits
mailing list