<div dir="ltr">Seems like a reasonable request.<div>I'll poke my head around in the tests tonight and see what I can do.</div><div>Thanks for the link.</div><div><br></div><div>/Eric</div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Fri, Aug 22, 2014 at 5:44 PM, Steve MacKenzie <span dir="ltr"><<a href="mailto:stevemac321@live.com" target="_blank">stevemac321@live.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div lang="EN-US" link="#0563C1" vlink="#954F72"><div><p class="MsoNormal">Greetings,<u></u><u></u></p><p class="MsoNormal">I would like to resume testing MSVC STL using the Libc++ tests.   <u></u><u></u></p><p class="MsoNormal">
<u></u> <u></u></p><p class="MsoNormal">Would it be OK to add noexcept specifiers to some of the test structs that are used as template params to std::atomic?   I believe this change is benign, so I am hoping you don’t care or require conditional blocks.    <u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p><p style="margin:0in;margin-bottom:.0001pt">Explanation from: <span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:black"><a href="https://connect.microsoft.com/VisualStudio/feedbackdetail/view/906973" target="_blank">https://connect.microsoft.com/VisualStudio/feedbackdetail/view/906973</a><u></u><u></u></span></p>
<p class="MsoNormal"><u></u> <u></u></p><p style="margin-right:0in;margin-bottom:0in;margin-left:27.0pt;margin-bottom:.0001pt"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:black">atomic() noexcept = default", so when the default constructor for<u></u><u></u></span></p>
<p style="margin-right:0in;margin-bottom:0in;margin-left:27.0pt;margin-bottom:.0001pt"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:black">std::atomic<T> is invoked for a type T that has a throwing constructor,<u></u><u></u></span></p>
<p style="margin-right:0in;margin-bottom:0in;margin-left:27.0pt;margin-bottom:.0001pt"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:black">the noexcept version is not found, which causes the compiler to emit<u></u><u></u></span></p>
<p style="margin-right:0in;margin-bottom:0in;margin-left:27.0pt;margin-bottom:.0001pt"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:black">an error message about the deleted function. To work around this,<u></u><u></u></span></p>
<p style="margin-right:0in;margin-bottom:0in;margin-left:27.0pt;margin-bottom:.0001pt"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:black">you need to declare the constructor of test_struct as noexcept.<u></u><u></u></span></p>
<p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">The affected tests pass using:<u></u><u></u></p><p class="MsoNormal">clang version 3.5.0 (trunk 212120)<span style="font-size:12.0pt"><u></u><u></u></span></p><p class="MsoNormal">
Target: x86_64-unknown-linux-gnu<u></u><u></u></p><p class="MsoNormal">Thread model: posix<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">Thanks,<u></u><u></u></p><p class="MsoNormal">Steve MacKenzie<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal"><u></u> <u></u></p></div></div><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>