[PATCH] test-suite: Fix name clash 'log2'
David Blaikie
dblaikie at gmail.com
Fri Jan 24 09:43:43 PST 2014
On Fri, Jan 24, 2014 at 9:31 AM, robert lytton <robert at xmos.com> wrote:
>
> It seems the xcore cmath.h file is buggy.
> We are exposing the math.h macro.
> Looking at an upstream version of the math.h header, the log2 macro has
> an __cplusplus guard around it.
> This is the correct fix.
>
Great - I assume the same is true for the "infinity" code you have out for
review too? You can cancel both these CRs, then?
>
> Patch is wrong.
>
> http://llvm-reviews.chandlerc.com/D2618
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140124/75f3838a/attachment.html>
More information about the llvm-commits
mailing list