<div dir="ltr"><br><div>Ping. :-)</div><div><br></div><div>It's just a test improvement patch so it shouldn't be controversial.</div><div><br></div><div>Cheers,</div><div>    Dario Domizioli</div><div>    SN Systems - Sony Computer Entertainment Group</div>
<div><br></div><div><br></div><div><br></div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 15 July 2014 16:45, Dario Domizioli <span dir="ltr"><<a href="mailto:dario.domizioli@gmail.com" target="_blank">dario.domizioli@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hello cfe-dev,</div><div><br></div><div>I am attaching a patch which improves the tests for #pragma clang optimize off/on. </div>
<div><br></div><div>It's just a few minor points, but we were missing some coverage. Specifically:</div>
<div>- A few more test cases are added to the SemaCXX test, so that they verify the behaviour of multiple "off" regions in the same file.</div><div>- One of those test cases also documents the fact that an "off" region can fall through the end of a #included file.</div>

<div>- A comment is added in the PCH test so that it's clearer how it works.</div><div><br></div><div>No functional change - only test changes.</div><div><br></div><div>Cheers,</div><div>    Dario Domizioli</div><div>

    SN Systems - Sony Computer Entertainment Group</div></div>
</blockquote></div><br></div>