r266186 - Enable support for __float128 in Clang

Hans Wennborg via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 14 18:07:58 PDT 2016


On Wed, Apr 13, 2016 at 2:49 AM, Nemanja Ivanovic via cfe-commits
<cfe-commits at lists.llvm.org> wrote:
> Author: nemanjai
> Date: Wed Apr 13 04:49:45 2016
> New Revision: 266186
>
> URL: http://llvm.org/viewvc/llvm-project?rev=266186&view=rev
> Log:
> Enable support for __float128 in Clang
>
> This patch corresponds to review:
> http://reviews.llvm.org/D15120
>
> It adds support for the __float128 keyword, literals and a target feature to
> enable it. This support is disabled by default on all targets and any target
> that has support for this type is free to add it.
>
> Based on feedback that I've received from target maintainers, this appears to
> be the right thing for most targets. I have not heard from the maintainers of
> X86 which I believe supports this type. I will subsequently investigate the
> impact of enabling this on X86.

We're seeing build errors when targeting Android, which I think may be
caused by this:

[...]
In file included from ../../v8/src/base/functional.cc:11:
In file included from ../../v8/src/base/functional.h:13:
In file included from
/usr/lib/gcc/x86_64-linux-gnu/4.8/../../../../include/c++/4.8/functional:55:
In file included from
/usr/lib/gcc/x86_64-linux-gnu/4.8/../../../../include/c++/4.8/tuple:38:
In file included from
/usr/lib/gcc/x86_64-linux-gnu/4.8/../../../../include/c++/4.8/utility:70:
In file included from
/usr/lib/gcc/x86_64-linux-gnu/4.8/../../../../include/c++/4.8/bits/stl_pair.h:59:
In file included from
/usr/lib/gcc/x86_64-linux-gnu/4.8/../../../../include/c++/4.8/bits/move.h:57:
/usr/lib/gcc/x86_64-linux-gnu/4.8/../../../../include/c++/4.8/type_traits:269:39:
error: __float128 is not supported on this target
    struct __is_floating_point_helper<__float128>
                                      ^

(From https://build.chromium.org/p/tryserver.chromium.android/builders/linux_android_rel_ng/builds/54128/steps/compile%20%28with%20patch%29/logs/stdio)

Any idea what might be breaking here?

Thanks,
Hans


More information about the cfe-commits mailing list