[PATCH] D45060: Allow target to handle required features for TARGET_BUILTINs.

Justin Lebar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 29 16:27:10 PDT 2018


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





More information about the cfe-commits mailing list