From: Eric Fiselier <eric@efcs.ca>
Date: Fri, 14 Oct 2016 21:30:35 +0000 (+0000)
Subject: Clarify XFAIL comments
X-Git-Tag: llvmorg-4.0.0-rc1~7117
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=797cb4f64614bf14c9dfccf67248718b716e1496;p=platform%2Fupstream%2Fllvm.git

Clarify XFAIL comments

llvm-svn: 284282
---

diff --git a/libcxx/test/std/language.support/support.dynamic/new.delete/new.delete.array/delete_align_val_t_replace.pass.cpp b/libcxx/test/std/language.support/support.dynamic/new.delete/new.delete.array/delete_align_val_t_replace.pass.cpp
index b8a5f7c..62ffa3f 100644
--- a/libcxx/test/std/language.support/support.dynamic/new.delete/new.delete.array/delete_align_val_t_replace.pass.cpp
+++ b/libcxx/test/std/language.support/support.dynamic/new.delete/new.delete.array/delete_align_val_t_replace.pass.cpp
@@ -17,7 +17,7 @@
 // None of the current GCC compilers support this.
 // XFAIL: gcc-4, gcc-5, gcc-6
 
-// UBSAN replaces certain new/delete functions which makes this test fail
+// TODO Investigate why UBSAN prevents new from calling our replacement.
 // XFAIL: ubsan
 
 #include <new>
@@ -60,6 +60,7 @@ struct alignas(std::max_align_t) B {};
 
 int main()
 {
+    reset();
     {
         B *x = new B[2];
         assert(0 == unsized_delete_called);
diff --git a/libcxx/test/std/language.support/support.dynamic/new.delete/new.delete.single/delete_align_val_t_replace.pass.cpp b/libcxx/test/std/language.support/support.dynamic/new.delete/new.delete.single/delete_align_val_t_replace.pass.cpp
index 02522d0..67a5191 100644
--- a/libcxx/test/std/language.support/support.dynamic/new.delete/new.delete.single/delete_align_val_t_replace.pass.cpp
+++ b/libcxx/test/std/language.support/support.dynamic/new.delete/new.delete.single/delete_align_val_t_replace.pass.cpp
@@ -17,7 +17,7 @@
 // None of the current GCC compilers support this.
 // XFAIL: gcc-4, gcc-5, gcc-6
 
-// UBSAN replaces certain new/delete functions which makes this test fail
+// TODO Investigate why UBSAN prevents new from calling our replacement.
 // XFAIL: ubsan
 
 #include <new>
@@ -60,6 +60,7 @@ struct alignas(std::max_align_t) B {};
 
 int main()
 {
+    reset();
     {
         B *x = new B;
         assert(0 == unsized_delete_called);