jlebar accepted this revision. jlebar added inline comments. This revision is now accepted and ready to land.
================ Comment at: clang/include/clang/Basic/TargetInfo.h:933 + /// FeatureMap. Targets may override this to provide custom handling of + /// required features. + virtual Optional<bool> ---------------- It's not totally obvious to me, can we explain why it's a tristate? https://reviews.llvm.org/D45060 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
