[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