
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">

<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>

</head>

<body dir="ltr">

<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">

I don't have context for the original issue, but our test selector does inspect some of the LIT control comments. Specifically, it skips tests with any of the comments:</div>

<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">

<span><br>

</span></div>

<blockquote style="margin: 0 0 0 40px; border: none; padding: 0px;">

<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">

<span>// REQUIRES: c++11</span></div>

<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">

<div>// REQUIRES: c++11 || c++14</div>

</div>

<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">

<div>// REQUIRES: c++98 || c++03</div>

</div>

<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">

<div>// REQUIRES: c++98 || c++03 || c++11 || c++14</div>

</div>

<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">

<div>// UNSUPPORTED: c++14, c++17, c++2a</div>

</div>

</blockquote>

<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">

<span></span><br>

</div>

<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">

So if were talking about something that doesn't touch those, it won't affect our typical usage.</div>

<div id="signature">

<div id="divtagdefaultwrapper" style="color:rgb(0,0,0); font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; background-color:rgb(255,255,255)">

<p style="margin-top: 0px; margin-bottom: 0px;"><br>

</p>

</div>

</div>

<div id="appendonsend"></div>

<hr style="display:inline-block;width:98%" tabindex="-1">

<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Stephan T. Lavavej<br>

<b>Sent:</b> Monday, December 10, 2018 15:43<br>

<b>To:</b> Jonathan Wakely; ldionne@apple.com<br>

<b>Cc:</b> Richard Smith; Marshall Clow; libcxx-dev@lists.llvm.org; Billy O'Neal (VC LIBS); Casey Carter<br>

<b>Subject:</b> RE: [libcxx-dev] How to check for a feature-test macro</font>

<div> </div>

</div>

<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">

<div class="PlainText">(+Casey who also works on this)<br>

<br>

When running libcxx's tests against MSVC's STL, we have an MSVC-internal skipped_tests.txt (full content available if requested) which says:<br>

<br>

# *** MISSING STL FEATURES ***<br>

# C++20 P0122R7 "<span>"<br>

upstream\test\std\containers\views\types.pass.cpp<br>

[...]<br>

<br>

# C++20 P0355R7 "<chrono> Calendars And Time Zones"<br>

upstream\test\std\utilities\time\days.pass.cpp<br>

[...]<br>

<br>

Updating this requires minimal effort. It's more problematic when existing tests are modified to test new features, since we have to skip the entire test even if only a small part is affected.<br>

<br>

Guarding tests with feature-test macros in their source code (as opposed to "the LIT level", which I believe is machinery that we don't use) would be ideal, I think.<br>

<br>

The annoying thing is when features are added without corresponding feature-test macros. There's a bunch of those right now. We'll probably need the skip-tests approach for those.<br>

<br>

STL<br>

<br>

-----Original Message-----<br>

From: Jonathan Wakely <cxx@kayari.org> <br>

Sent: Monday, December 10, 2018 12:10 PM<br>

To: ldionne@apple.com<br>

Cc: Richard Smith <richard@metafoo.co.uk>; Marshall Clow <mclow.lists@gmail.com>; libcxx-dev@lists.llvm.org; Stephan T. Lavavej <stl@exchange.microsoft.com>; Billy O'Neal (VC LIBS) <bion@microsoft.com><br>

Subject: Re: [libcxx-dev] How to check for a feature-test macro<br>

<br>

On Mon, 10 Dec 2018 at 19:47, Louis Dionne wrote:<br>

> We should also see how other implementations that use our test suite feel about this (CCing them).<br>

<br>

I only use your tests a few times a year, run by hand. If I get a load of FAILs because I've not implemented something yet and you have, that's not a problem. I don't expect to get 100% green when I run your tests.<br>

</div>

</span></font></div>

</body>

</html>