<div dir="ltr">FTR I agree with this approach, I don't see why we should treat these macros specially.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 21, 2014 at 12:35 PM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Tue, Oct 21, 2014 at 11:31 AM, JF Bastien <<a href="mailto:jfb@chromium.org">jfb@chromium.org</a>> wrote:<br>
>> The compiler has documented sizes for fundamental types which can be<br>
>> used to define the types in this file. Eg)<br>
>><br>
>> typedef signed char        int8_t;<br>
>> typedef short              int16_t;<br>
>> typedef int                int32_t;<br>
>><br>
>> Nothing says we cannot provide this in similar fashion for our<br>
>> stdint.h; though, as I pointed out, that could make this file *very*<br>
>> ugly with a bunch of target-specific #ifs everywhere to ensure the<br>
>> proper definitions of *int*_t and friends. I can understand not<br>
>> wanting to do that (especially since the compiler already has this<br>
>> information internally).<br>
><br>
><br>
> The compiler guaranteeing something is different from that guarantee being<br>
> in the language. C and C++ standard libraries like newlib target multiple<br>
> compilers and can't rely on what LLVM guarantees, so they either have to use<br>
> configure and ask the compiler by compiling a program, or just straight up<br>
> ask the compiler when it provides nice macros.<br>
<br>
</div></div>Since others seem to be strongly in favor of (2), I will favor it as<br>
well. I have made commit r220312 which implements (2), and brings the<br>
MSVC compat bots back to being green.<br>
<div class="HOEnZb"><div class="h5"><br>
~Aaron<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>
</div></div></blockquote></div><br></div>