[libcxx-commits] [PATCH] D77563: [libc++] Move C++17 check around optional only in ObjC is_scalar test.

[Zoe Carver via Phabricator via libcxx-commits]

zoecarver updated this revision to Diff 255413.
zoecarver added a comment.

- Remove std::optional test altogether because it's tested elsewhere


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77563/new/

https://reviews.llvm.org/D77563

Files:
  libcxx/test/libcxx/type_traits/is_scalar.objc.pass.mm


Index: libcxx/test/libcxx/type_traits/is_scalar.objc.pass.mm
===================================================================
--- libcxx/test/libcxx/type_traits/is_scalar.objc.pass.mm
+++ libcxx/test/libcxx/type_traits/is_scalar.objc.pass.mm
@@ -17,7 +17,6 @@
 // Make sure we report that blocks are scalar types.
 
 #include <type_traits>
-#include <optional>
 
 struct Foo { };
 template <int> struct Arg { };
@@ -35,6 +34,5 @@
 
 
 int main(int, char**) {
-    std::optional<Foo (^)(Arg<0>)> opt; (void)opt;
     return 0;
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D77563.255413.patch
Type: text/x-patch
Size: 532 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20200406/d8273181/attachment.bin>