<div dir="ltr">Hi Jyotsna,<div><br></div><div>You are right this is poor behavior in the tests (which shouldn't ever depend on the system headers).</div><div><br></div><div>I believe the c11.c test should be fixed here:</div>

<div>  <a href="http://llvm.org/viewvc/llvm-project?view=revision&revision=179723" target="_blank">http://llvm.org/viewvc/llvm-project?view=revision&revision=179723</a></div><div><br></div><div style>Is that not the case?</div>
<div style><br></div><div> - Daniel</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Apr 17, 2013 at 12:19 PM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</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 class="im">On Wed, Apr 17, 2013 at 10:39 AM, Jyotsna Verma <span dir="ltr"><<a href="mailto:jverma@codeaurora.org" target="_blank">jverma@codeaurora.org</a>></span> wrote:<br>
</div><div class="gmail_extra"><div class="gmail_quote"><div class="im">

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi Daniel/Richard,<br>
<br>
I'm seeing c11.c and cxx11.cpp tests failing on Hexagon with these two<br>
changes - r179427 and r179419.<br>
<br>
After your changes, we include stdint.h file (from lib/Headers) in both<br>
these tests. However,  if  __STDC_HOSTED__ macro is defined,<br>
lib/Headers/stdint.h includes system stdint.h file which includes features.h<br>
-> gnu/stubs-32.h. Since, we don't have stubs-32.h header file available for<br>
Hexagon, it causes the tests to fail.<br>
<br>
Please help me understand why the tests were modified to include stdint.h<br>
file and how to fix them for Hexagon.</blockquote><div><br></div></div><div>These tests are testing the behavior of our <stdint.h> implementation. It would certainly make sense to make them avoid including a system-provided <stdint.h>. That will probably require somewhat different fixes for each test:</div>


<div><br></div><div>I think (perhaps Daniel can confirm) that the c11.c test just wants to test the contents of our own <stdint.h>, so modifying the test's RUN: line so that the system-provided <stdint.h> is not on the search path would suffice.</div>


<div><br></div><div>For the cxx11.c test, what we're checking is that our builtin <stdint.h> correctly works around <a href="http://sourceware.org/PR15366" target="_blank">http://sourceware.org/PR15366</a> -- though a better (non-system-dependent) test here would be to craft a "system header" <stdint.h> with the C99-specified behavior and put that in the system headers include path for the test.</div>


</div></div></div>
</blockquote></div><br></div>