<div dir="ltr">Our bots seem happier now. Thanks for the quick fix!</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Dec 6, 2018 at 1:12 PM Louis Dionne <<a href="mailto:ldionne@apple.com">ldionne@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="overflow-wrap: break-word;"><br><div><br><blockquote type="cite"><div>On Dec 6, 2018, at 10:50, Hans Wennborg <<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>> wrote:</div><br class="gmail-m_5507782158956407804Apple-interchange-newline"><div><span style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none;float:none;display:inline">Hi Louis,</span><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><span style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none;float:none;display:inline">These tests now fail for us (they're xfailed and unexpectedly pass)</span><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><span style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none;float:none;display:inline">when building libc++ for Chromium.</span><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><span style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none;float:none;display:inline">We're building with MACOSX_DEPLOYMENT_TARGET=10.7, with AppleClang</span><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><span style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none;float:none;display:inline">9.0.0.9000037. Should the xfail checks be updated to take this into</span><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><span style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none;float:none;display:inline">account?</span><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"></div></blockquote><div><br></div><div>Thanks for the heads up and sorry about that. This should be fixed in r348507.</div><div><br></div><div>These aligned allocation tests are incredibly tricky because they are unlike any other availability test we have. They depend on the compiler and standard library version used in subtle ways, and we don't have quite enough expressiveness in LIT to say what we want (that the tests should XFAIL for all apple-clang less than or equal to 10).</div><div><br></div><div>Please LMK if you still see failures.</div><div><br></div><div>Louis</div><div><br></div><br><blockquote type="cite"><div><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><span style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none;float:none;display:inline">On Mon, Nov 26, 2018 at 8:32 PM Louis Dionne via libcxx-commits</span><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><span style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none;float:none;display:inline"><</span><a href="mailto:libcxx-commits@lists.llvm.org" style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px" target="_blank">libcxx-commits@lists.llvm.org</a><span style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none;float:none;display:inline">> wrote:</span><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><blockquote type="cite" style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><br><br><br><br>---------- Forwarded message ----------<br>From: Louis Dionne <<a href="mailto:ldionne@apple.com" target="_blank">ldionne@apple.com</a>><br>To:<span class="gmail-m_5507782158956407804Apple-converted-space"> </span><a href="mailto:libcxx-commits@lists.llvm.org" target="_blank">libcxx-commits@lists.llvm.org</a><br>Cc:<br>Bcc:<br>Date: Mon, 26 Nov 2018 19:30:08 -0000<br>Subject: [libcxx] r347585 - [libcxx] Fix XFAILs for aligned allocation tests<br>Author: ldionne<br>Date: Mon Nov 26 11:30:08 2018<br>New Revision: 347585<br><br>URL: <a href="http://llvm.org/viewvc/llvm-project?rev=347585&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=347585&view=rev</a><br>Log:<br>[libcxx] Fix XFAILs for aligned allocation tests<br><br>In r339743, I marked several aligned allocation tests as downright<br>unsupported on macosx in an attempt to unbreak the build. It turns<br>out that marking them as unuspported whenever we're on OS X is way<br>too coarse grained. This commit marks the tests as XFAIL with more<br>granularity.<br><br>Modified:<br>   libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/delete_align_val_t_replace.pass.cpp<br>   libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/new_align_val_t.pass.cpp<br>   libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/new_align_val_t_nothrow.pass.cpp<br>   libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/new_align_val_t_nothrow_replace.pass.cpp<br>   libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/delete_align_val_t_replace.pass.cpp<br>   libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_align_val_t.pass.cpp<br>   libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_align_val_t_nothrow.pass.cpp<br>   libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_align_val_t_nothrow_replace.pass.cpp<br><br>Modified: libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/delete_align_val_t_replace.pass.cpp<br>URL: <a href="http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/delete_align_val_t_replace.pass.cpp?rev=347585&r1=347584&r2=347585&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/delete_align_val_t_replace.pass.cpp?rev=347585&r1=347584&r2=347585&view=diff</a><br>==============================================================================<br>--- libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/delete_align_val_t_replace.pass.cpp (original)<br>+++ libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/delete_align_val_t_replace.pass.cpp Mon Nov 26 11:30:08 2018<br>@@ -17,22 +17,25 @@<br>// None of the current GCC compilers support this.<br>// UNSUPPORTED: gcc-5, gcc-6<br><br>-// dylibs shipped before macosx10.13 do not provide aligned allocation, so that's a link error<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.12<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.11<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.10<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.9<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.8<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.7<br>+// Aligned allocation was not provided before macosx10.12 and as a result we<br>+// get availability errors when the deployment target is older than macosx10.13.<br>+// However, AppleClang 10 (and older) don't trigger availability errors.<br>+// XFAIL: !apple-clang-10 && availability=macosx10.12<br>+// XFAIL: !apple-clang-10 && availability=macosx10.11<br>+// XFAIL: !apple-clang-10 && availability=macosx10.10<br>+// XFAIL: !apple-clang-10 && availability=macosx10.9<br>+// XFAIL: !apple-clang-10 && availability=macosx10.8<br>+// XFAIL: !apple-clang-10 && availability=macosx10.7<br><br>-// Using aligned allocation functions is a compiler error when deploying to<br>-// platforms older than macosx10.13<br>-// UNSUPPORTED: macosx10.12<br>-// UNSUPPORTED: macosx10.11<br>-// UNSUPPORTED: macosx10.10<br>-// UNSUPPORTED: macosx10.9<br>-// UNSUPPORTED: macosx10.8<br>-// UNSUPPORTED: macosx10.7<br>+// On AppleClang 10 (and older), instead of getting an availability failure<br>+// like above, we get a link error when we link against a dylib that does<br>+// not export the aligned allocation functions.<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.12<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.11<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.10<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.9<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.8<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.7<br><br>// On Windows libc++ doesn't provide its own definitions for new/delete<br>// but instead depends on the ones in VCRuntime. However VCRuntime does not<br><br>Modified: libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/new_align_val_t.pass.cpp<br>URL: <a href="http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/new_align_val_t.pass.cpp?rev=347585&r1=347584&r2=347585&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/new_align_val_t.pass.cpp?rev=347585&r1=347584&r2=347585&view=diff</a><br>==============================================================================<br>--- libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/new_align_val_t.pass.cpp (original)<br>+++ libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/new_align_val_t.pass.cpp Mon Nov 26 11:30:08 2018<br>@@ -15,22 +15,25 @@<br>// FIXME change this to XFAIL.<br>// UNSUPPORTED: no-aligned-allocation && !gcc<br><br>-// dylibs shipped before macosx10.13 do not provide aligned allocation, so that's a link error<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.12<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.11<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.10<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.9<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.8<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.7<br>+// Aligned allocation was not provided before macosx10.12 and as a result we<br>+// get availability errors when the deployment target is older than macosx10.13.<br>+// However, AppleClang 10 (and older) don't trigger availability errors.<br>+// XFAIL: !apple-clang-10 && availability=macosx10.12<br>+// XFAIL: !apple-clang-10 && availability=macosx10.11<br>+// XFAIL: !apple-clang-10 && availability=macosx10.10<br>+// XFAIL: !apple-clang-10 && availability=macosx10.9<br>+// XFAIL: !apple-clang-10 && availability=macosx10.8<br>+// XFAIL: !apple-clang-10 && availability=macosx10.7<br><br>-// Using aligned allocation functions is a compiler error when deploying to<br>-// platforms older than macosx10.13<br>-// UNSUPPORTED: macosx10.12<br>-// UNSUPPORTED: macosx10.11<br>-// UNSUPPORTED: macosx10.10<br>-// UNSUPPORTED: macosx10.9<br>-// UNSUPPORTED: macosx10.8<br>-// UNSUPPORTED: macosx10.7<br>+// On AppleClang 10 (and older), instead of getting an availability failure<br>+// like above, we get a link error when we link against a dylib that does<br>+// not export the aligned allocation functions.<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.12<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.11<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.10<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.9<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.8<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.7<br><br>// On Windows libc++ doesn't provide its own definitions for new/delete<br>// but instead depends on the ones in VCRuntime. However VCRuntime does not<br><br>Modified: libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/new_align_val_t_nothrow.pass.cpp<br>URL: <a href="http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/new_align_val_t_nothrow.pass.cpp?rev=347585&r1=347584&r2=347585&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/new_align_val_t_nothrow.pass.cpp?rev=347585&r1=347584&r2=347585&view=diff</a><br>==============================================================================<br>--- libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/new_align_val_t_nothrow.pass.cpp (original)<br>+++ libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/new_align_val_t_nothrow.pass.cpp Mon Nov 26 11:30:08 2018<br>@@ -15,22 +15,25 @@<br>// FIXME turn this into an XFAIL<br>// UNSUPPORTED: no-aligned-allocation && !gcc<br><br>-// dylibs shipped before macosx10.13 do not provide aligned allocation, so that's a link error<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.12<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.11<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.10<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.9<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.8<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.7<br>+// Aligned allocation was not provided before macosx10.12 and as a result we<br>+// get availability errors when the deployment target is older than macosx10.13.<br>+// However, AppleClang 10 (and older) don't trigger availability errors.<br>+// XFAIL: !apple-clang-10 && availability=macosx10.12<br>+// XFAIL: !apple-clang-10 && availability=macosx10.11<br>+// XFAIL: !apple-clang-10 && availability=macosx10.10<br>+// XFAIL: !apple-clang-10 && availability=macosx10.9<br>+// XFAIL: !apple-clang-10 && availability=macosx10.8<br>+// XFAIL: !apple-clang-10 && availability=macosx10.7<br><br>-// Using aligned allocation functions is a compiler error when deploying to<br>-// platforms older than macosx10.13<br>-// UNSUPPORTED: macosx10.12<br>-// UNSUPPORTED: macosx10.11<br>-// UNSUPPORTED: macosx10.10<br>-// UNSUPPORTED: macosx10.9<br>-// UNSUPPORTED: macosx10.8<br>-// UNSUPPORTED: macosx10.7<br>+// On AppleClang 10 (and older), instead of getting an availability failure<br>+// like above, we get a link error when we link against a dylib that does<br>+// not export the aligned allocation functions.<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.12<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.11<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.10<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.9<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.8<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.7<br><br>// On Windows libc++ doesn't provide its own definitions for new/delete<br>// but instead depends on the ones in VCRuntime. However VCRuntime does not<br><br>Modified: libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/new_align_val_t_nothrow_replace.pass.cpp<br>URL: <a href="http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/new_align_val_t_nothrow_replace.pass.cpp?rev=347585&r1=347584&r2=347585&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/new_align_val_t_nothrow_replace.pass.cpp?rev=347585&r1=347584&r2=347585&view=diff</a><br>==============================================================================<br>--- libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/new_align_val_t_nothrow_replace.pass.cpp (original)<br>+++ libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.array/new_align_val_t_nothrow_replace.pass.cpp Mon Nov 26 11:30:08 2018<br>@@ -10,24 +10,25 @@<br>// UNSUPPORTED: c++98, c++03, c++11, c++14<br>// UNSUPPORTED: sanitizer-new-delete<br><br>-// dylibs shipped before macosx10.13 do not provide aligned allocation, so our<br>-// custom aligned allocation functions are not called and the test fails<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.12<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.11<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.10<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.9<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.8<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.7<br>+// Aligned allocation was not provided before macosx10.12 and as a result we<br>+// get availability errors when the deployment target is older than macosx10.13.<br>+// However, AppleClang 10 (and older) don't trigger availability errors.<br>+// XFAIL: !apple-clang-10 && availability=macosx10.12<br>+// XFAIL: !apple-clang-10 && availability=macosx10.11<br>+// XFAIL: !apple-clang-10 && availability=macosx10.10<br>+// XFAIL: !apple-clang-10 && availability=macosx10.9<br>+// XFAIL: !apple-clang-10 && availability=macosx10.8<br>+// XFAIL: !apple-clang-10 && availability=macosx10.7<br><br>-// Our custom aligned allocation functions are not called when deploying to<br>-// platforms older than macosx10.13, since those platforms don't support<br>-// aligned allocation.<br>-// UNSUPPORTED: macosx10.12<br>-// UNSUPPORTED: macosx10.11<br>-// UNSUPPORTED: macosx10.10<br>-// UNSUPPORTED: macosx10.9<br>-// UNSUPPORTED: macosx10.8<br>-// UNSUPPORTED: macosx10.7<br>+// On AppleClang 10 (and older), instead of getting an availability failure<br>+// like above, we get a link error when we link against a dylib that does<br>+// not export the aligned allocation functions.<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.12<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.11<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.10<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.9<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.8<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.7<br><br>// XFAIL: no-aligned-allocation && !gcc<br><br><br>Modified: libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/delete_align_val_t_replace.pass.cpp<br>URL: <a href="http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/delete_align_val_t_replace.pass.cpp?rev=347585&r1=347584&r2=347585&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/delete_align_val_t_replace.pass.cpp?rev=347585&r1=347584&r2=347585&view=diff</a><br>==============================================================================<br>--- libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/delete_align_val_t_replace.pass.cpp (original)<br>+++ libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/delete_align_val_t_replace.pass.cpp Mon Nov 26 11:30:08 2018<br>@@ -16,22 +16,25 @@<br>// None of the current GCC compilers support this.<br>// UNSUPPORTED: gcc-5, gcc-6<br><br>-// dylibs shipped before macosx10.13 do not provide aligned allocation, so that's a link error<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.12<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.11<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.10<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.9<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.8<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.7<br>+// Aligned allocation was not provided before macosx10.12 and as a result we<br>+// get availability errors when the deployment target is older than macosx10.13.<br>+// However, AppleClang 10 (and older) don't trigger availability errors.<br>+// XFAIL: !apple-clang-10 && availability=macosx10.12<br>+// XFAIL: !apple-clang-10 && availability=macosx10.11<br>+// XFAIL: !apple-clang-10 && availability=macosx10.10<br>+// XFAIL: !apple-clang-10 && availability=macosx10.9<br>+// XFAIL: !apple-clang-10 && availability=macosx10.8<br>+// XFAIL: !apple-clang-10 && availability=macosx10.7<br><br>-// Using aligned allocation functions is a compiler error when deploying to<br>-// platforms older than macosx10.13<br>-// UNSUPPORTED: macosx10.12<br>-// UNSUPPORTED: macosx10.11<br>-// UNSUPPORTED: macosx10.10<br>-// UNSUPPORTED: macosx10.9<br>-// UNSUPPORTED: macosx10.8<br>-// UNSUPPORTED: macosx10.7<br>+// On AppleClang 10 (and older), instead of getting an availability failure<br>+// like above, we get a link error when we link against a dylib that does<br>+// not export the aligned allocation functions.<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.12<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.11<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.10<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.9<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.8<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.7<br><br>// On Windows libc++ doesn't provide its own definitions for new/delete<br>// but instead depends on the ones in VCRuntime. However VCRuntime does not<br><br>Modified: libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_align_val_t.pass.cpp<br>URL: <a href="http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_align_val_t.pass.cpp?rev=347585&r1=347584&r2=347585&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_align_val_t.pass.cpp?rev=347585&r1=347584&r2=347585&view=diff</a><br>==============================================================================<br>--- libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_align_val_t.pass.cpp (original)<br>+++ libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_align_val_t.pass.cpp Mon Nov 26 11:30:08 2018<br>@@ -9,22 +9,25 @@<br><br>// UNSUPPORTED: c++98, c++03, c++11, c++14<br><br>-// dylibs shipped before macosx10.13 do not provide aligned allocation, so that's a link error<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.12<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.11<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.10<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.9<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.8<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.7<br>+// Aligned allocation was not provided before macosx10.12 and as a result we<br>+// get availability errors when the deployment target is older than macosx10.13.<br>+// However, AppleClang 10 (and older) don't trigger availability errors.<br>+// XFAIL: !apple-clang-10 && availability=macosx10.12<br>+// XFAIL: !apple-clang-10 && availability=macosx10.11<br>+// XFAIL: !apple-clang-10 && availability=macosx10.10<br>+// XFAIL: !apple-clang-10 && availability=macosx10.9<br>+// XFAIL: !apple-clang-10 && availability=macosx10.8<br>+// XFAIL: !apple-clang-10 && availability=macosx10.7<br><br>-// Using aligned allocation functions is a compiler error when deploying to<br>-// platforms older than macosx10.13<br>-// UNSUPPORTED: macosx10.12<br>-// UNSUPPORTED: macosx10.11<br>-// UNSUPPORTED: macosx10.10<br>-// UNSUPPORTED: macosx10.9<br>-// UNSUPPORTED: macosx10.8<br>-// UNSUPPORTED: macosx10.7<br>+// On AppleClang 10 (and older), instead of getting an availability failure<br>+// like above, we get a link error when we link against a dylib that does<br>+// not export the aligned allocation functions.<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.12<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.11<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.10<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.9<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.8<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.7<br><br>// asan and msan will not call the new handler.<br>// UNSUPPORTED: sanitizer-new-delete<br><br>Modified: libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_align_val_t_nothrow.pass.cpp<br>URL: <a href="http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_align_val_t_nothrow.pass.cpp?rev=347585&r1=347584&r2=347585&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_align_val_t_nothrow.pass.cpp?rev=347585&r1=347584&r2=347585&view=diff</a><br>==============================================================================<br>--- libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_align_val_t_nothrow.pass.cpp (original)<br>+++ libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_align_val_t_nothrow.pass.cpp Mon Nov 26 11:30:08 2018<br>@@ -9,22 +9,25 @@<br><br>// UNSUPPORTED: c++98, c++03, c++11, c++14<br><br>-// dylibs shipped before macosx10.13 do not provide aligned allocation, so that's a link error<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.12<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.11<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.10<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.9<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.8<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.7<br>+// Aligned allocation was not provided before macosx10.12 and as a result we<br>+// get availability errors when the deployment target is older than macosx10.13.<br>+// However, AppleClang 10 (and older) don't trigger availability errors.<br>+// XFAIL: !apple-clang-10 && availability=macosx10.12<br>+// XFAIL: !apple-clang-10 && availability=macosx10.11<br>+// XFAIL: !apple-clang-10 && availability=macosx10.10<br>+// XFAIL: !apple-clang-10 && availability=macosx10.9<br>+// XFAIL: !apple-clang-10 && availability=macosx10.8<br>+// XFAIL: !apple-clang-10 && availability=macosx10.7<br><br>-// Using aligned allocation functions is a compiler error when deploying to<br>-// platforms older than macosx10.13<br>-// UNSUPPORTED: macosx10.12<br>-// UNSUPPORTED: macosx10.11<br>-// UNSUPPORTED: macosx10.10<br>-// UNSUPPORTED: macosx10.9<br>-// UNSUPPORTED: macosx10.8<br>-// UNSUPPORTED: macosx10.7<br>+// On AppleClang 10 (and older), instead of getting an availability failure<br>+// like above, we get a link error when we link against a dylib that does<br>+// not export the aligned allocation functions.<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.12<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.11<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.10<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.9<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.8<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.7<br><br>// asan and msan will not call the new handler.<br>// UNSUPPORTED: sanitizer-new-delete<br><br>Modified: libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_align_val_t_nothrow_replace.pass.cpp<br>URL: <a href="http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_align_val_t_nothrow_replace.pass.cpp?rev=347585&r1=347584&r2=347585&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_align_val_t_nothrow_replace.pass.cpp?rev=347585&r1=347584&r2=347585&view=diff</a><br>==============================================================================<br>--- libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_align_val_t_nothrow_replace.pass.cpp (original)<br>+++ libcxx/trunk/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_align_val_t_nothrow_replace.pass.cpp Mon Nov 26 11:30:08 2018<br>@@ -10,24 +10,25 @@<br>// UNSUPPORTED: c++98, c++03, c++11, c++14<br>// UNSUPPORTED: sanitizer-new-delete<br><br>-// dylibs shipped before macosx10.13 do not provide aligned allocation, so our<br>-// custom aligned allocation functions are not called and the test fails<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.12<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.11<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.10<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.9<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.8<br>-// UNSUPPORTED: with_system_cxx_lib=macosx10.7<br>+// Aligned allocation was not provided before macosx10.12 and as a result we<br>+// get availability errors when the deployment target is older than macosx10.13.<br>+// However, AppleClang 10 (and older) don't trigger availability errors.<br>+// XFAIL: !apple-clang-10 && availability=macosx10.12<br>+// XFAIL: !apple-clang-10 && availability=macosx10.11<br>+// XFAIL: !apple-clang-10 && availability=macosx10.10<br>+// XFAIL: !apple-clang-10 && availability=macosx10.9<br>+// XFAIL: !apple-clang-10 && availability=macosx10.8<br>+// XFAIL: !apple-clang-10 && availability=macosx10.7<br><br>-// Our custom aligned allocation functions are not called when deploying to<br>-// platforms older than macosx10.13, since those platforms don't support<br>-// aligned allocation.<br>-// UNSUPPORTED: macosx10.12<br>-// UNSUPPORTED: macosx10.11<br>-// UNSUPPORTED: macosx10.10<br>-// UNSUPPORTED: macosx10.9<br>-// UNSUPPORTED: macosx10.8<br>-// UNSUPPORTED: macosx10.7<br>+// On AppleClang 10 (and older), instead of getting an availability failure<br>+// like above, we get a link error when we link against a dylib that does<br>+// not export the aligned allocation functions.<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.12<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.11<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.10<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.9<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.8<br>+// XFAIL: apple-clang-10 && with_system_cxx_lib=macosx10.7<br><br>// NOTE: gcc doesn't provide -faligned-allocation flag to test for<br>// XFAIL: no-aligned-allocation && !gcc<br><br><br><br><br><br>---------- Forwarded message ----------<br>From: Louis Dionne via libcxx-commits <<a href="mailto:libcxx-commits@lists.llvm.org" target="_blank">libcxx-commits@lists.llvm.org</a>><br>To:<span class="gmail-m_5507782158956407804Apple-converted-space"> </span><a href="mailto:libcxx-commits@lists.llvm.org" target="_blank">libcxx-commits@lists.llvm.org</a><br>Cc:<br>Bcc:<br>Date: Mon, 26 Nov 2018 11:30:09 -0800 (PST)<br>Subject: [libcxx-commits] [libcxx] r347585 - [libcxx] Fix XFAILs for aligned allocation tests<br>_______________________________________________<br>libcxx-commits mailing list<br><a href="mailto:libcxx-commits@lists.llvm.org" target="_blank">libcxx-commits@lists.llvm.org</a><br><a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/libcxx-commits" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/libcxx-commits</a></blockquote></div></blockquote></div><br></div></blockquote></div>