[libcxx] r304106 - Correct XFAIL's in coroutine tests to only list ubsan

[Eric Fiselier via cfe-commits]

Author: ericwf
Date: Sun May 28 15:56:16 2017
New Revision: 304106

URL: http://llvm.org/viewvc/llvm-project?rev=304106&view=rev
Log:
Correct XFAIL's in coroutine tests to only list ubsan

Modified:
    libcxx/trunk/test/std/experimental/language.support/support.coroutines/end.to.end/bool_await_suspend.sh.cpp
    libcxx/trunk/test/std/experimental/language.support/support.coroutines/end.to.end/generator.sh.cpp

Modified: libcxx/trunk/test/std/experimental/language.support/support.coroutines/end.to.end/bool_await_suspend.sh.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/experimental/language.support/support.coroutines/end.to.end/bool_await_suspend.sh.cpp?rev=304106&r1=304105&r2=304106&view=diff
==============================================================================
--- libcxx/trunk/test/std/experimental/language.support/support.coroutines/end.to.end/bool_await_suspend.sh.cpp (original)
+++ libcxx/trunk/test/std/experimental/language.support/support.coroutines/end.to.end/bool_await_suspend.sh.cpp Sun May 28 15:56:16 2017
@@ -11,9 +11,8 @@
 // UNSUPPORTED: c++98, c++03, c++11
 // REQUIRES: fcoroutines-ts
 
-// These configurations run the tests with '-g', which hits a bug in Clangs
-// coroutine implementation.
-// XFAIL: asan, msan, ubsan, tsan
+// FIXME: When run under UBSAN this test hits an assertion inside Clang
+// XFAIL: ubsan
 
 // RUN: %build -fcoroutines-ts
 // RUN: %run
@@ -26,14 +25,15 @@ using namespace std::experimental;
 struct coro_t {
   struct promise_type {
     coro_t get_return_object() {
-      coroutine_handle<promise_type>{};
-      return {};
+      return coroutine_handle<promise_type>::from_promise(*this);
     }
     suspend_never initial_suspend() { return {}; }
     suspend_never final_suspend() { return {}; }
     void return_void(){}
     void unhandled_exception() {}
   };
+  coro_t(coroutine_handle<promise_type> hh) : h(hh) {}
+  coroutine_handle<promise_type> h;
 };
 
 struct NoSuspend {
@@ -64,7 +64,9 @@ coro_t g() {
 
 int main() {
   assert(!f_started && !f_resumed && !g_started && !g_resumed);
-  f();
+  auto fret = f();
+  assert(f_started && !f_resumed);
+  fret.h.destroy();
   assert(f_started && !f_resumed);
   g();
   assert(g_started && g_resumed);

Modified: libcxx/trunk/test/std/experimental/language.support/support.coroutines/end.to.end/generator.sh.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/experimental/language.support/support.coroutines/end.to.end/generator.sh.cpp?rev=304106&r1=304105&r2=304106&view=diff
==============================================================================
--- libcxx/trunk/test/std/experimental/language.support/support.coroutines/end.to.end/generator.sh.cpp (original)
+++ libcxx/trunk/test/std/experimental/language.support/support.coroutines/end.to.end/generator.sh.cpp Sun May 28 15:56:16 2017
@@ -11,9 +11,8 @@
 // UNSUPPORTED: c++98, c++03, c++11
 // REQUIRES: fcoroutines-ts
 
-// These configurations run the tests with '-g', which hits a bug in Clangs
-// coroutine implementation.
-// XFAIL: asan, msan, ubsan, tsan
+// FIXME: When run under UBSAN this test hits an assertion inside Clang
+// XFAIL: ubsan
 
 // RUN: %build -fcoroutines-ts
 // RUN: %run