EricWF accepted this revision. EricWF added a comment. This revision is now accepted and ready to land.
In https://reviews.llvm.org/D26934#604325, @smeenai wrote: > I'd prefer `#error`; that way we can eventually just drop the macro. Ideally > there wouldn't be too many users to begin with, since it was only introduced > two months ago, and it hasn't made its way to a release yet. If it hasn't even made it into a release then we probably don't need the `#error` at all. Feel free to commit with or w/o it. Either way this LGTM. https://reviews.llvm.org/D26934 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits