On Mon, Oct 19, 2015 at 12:39 PM, H.J. Lu <hjl.to...@gmail.com> wrote:
> On Mon, Jul 20, 2015 at 12:15 AM, Mikhail Maltsev <malts...@gmail.com> wrote:
>> On 07/17/2015 07:46 PM, Mike Stump wrote:
>>> On Jul 17, 2015, at 2:28 AM, Mikhail Maltsev <malts...@gmail.com> wrote:
>>>> The following code (reduced from wide-int.h) is rejected by Intel C++
>>>> Compiler (EDG-based):
>>>
>>> So, could you test this with the top of the tree compiler and file a bug
>>> report against g++ for it, if it seems to not work right. If that bug
>>> report
>>> is rejected, then I’d say file a bug report against clang and EDG.
>>
>> In addition to usual bootstrap+regtest, I also checked that build succeeds
>> with
>> GCC 4.3.6 (IIRC, this is now the minimal required version) as well as with
>> recent GCC snapshot used as stage 0. Committed as r225993.
>> I also filed this bugreport:
>> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66941
>>
>>>> I think that the warning is correct, and "template <>" should not be used
>>>> here. The attached patch should fix this issue. Bootstrapped and regtested
>>>> on x86_64-linux. OK for trunk?
>>>
>>> Ok. Does this need to go into the gcc-5 release branch as well? If so, ok
>>> there too. Thanks.
>> I think there is no need for it.
>
> It is also need for gcc-5. I am backporting it now.
>
This is what I checked into gcc-5-branch.
--
H.J.
From 4ae06c3dbe5fb2c4d345060b1ba9cd34b2dc7d37 Mon Sep 17 00:00:00 2001
From: miyuki <miyuki@138bc75d-0d04-0410-961f-82ee72b054a4>
Date: Mon, 20 Jul 2015 05:30:12 +0000
Subject: [PATCH] Fix partial specialization syntax of wide int traits.
gcc/
* wide-int.h (struct binary_traits): Fix partial specialization syntax.
(struct int_traits): Likewise.
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@225993 138bc75d-0d04-0410-961f-82ee72b054a4
---
gcc/ChangeLog | 8 ++++++++
gcc/wide-int.h | 10 ----------
2 files changed, 8 insertions(+), 10 deletions(-)
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 7fb0538..45ae071 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,11 @@
+2015-10-19 H.J. Lu <hongjiu...@intel.com>
+
+ Backport from mainline
+ 2015-07-20 Mikhail Maltsev <malts...@gmail.com>
+
+ * wide-int.h (struct binary_traits): Fix partial specialization syntax.
+ (struct int_traits): Likewise.
+
2015-10-16 Richard Sandiford <richard.sandif...@arm.com>
PR middle-end/66311
diff --git a/gcc/wide-int.h b/gcc/wide-int.h
index 46f4545..9a71c4f 100644
--- a/gcc/wide-int.h
+++ b/gcc/wide-int.h
@@ -365,21 +365,18 @@ namespace wi
inputs. Note that CONST_PRECISION and VAR_PRECISION cannot be
mixed, in order to give stronger type checking. When both inputs
are CONST_PRECISION, they must have the same precision. */
- template <>
template <typename T1, typename T2>
struct binary_traits <T1, T2, FLEXIBLE_PRECISION, FLEXIBLE_PRECISION>
{
typedef widest_int result_type;
};
- template <>
template <typename T1, typename T2>
struct binary_traits <T1, T2, FLEXIBLE_PRECISION, VAR_PRECISION>
{
typedef wide_int result_type;
};
- template <>
template <typename T1, typename T2>
struct binary_traits <T1, T2, FLEXIBLE_PRECISION, CONST_PRECISION>
{
@@ -389,14 +386,12 @@ namespace wi
<int_traits <T2>::precision> > result_type;
};
- template <>
template <typename T1, typename T2>
struct binary_traits <T1, T2, VAR_PRECISION, FLEXIBLE_PRECISION>
{
typedef wide_int result_type;
};
- template <>
template <typename T1, typename T2>
struct binary_traits <T1, T2, CONST_PRECISION, FLEXIBLE_PRECISION>
{
@@ -406,7 +401,6 @@ namespace wi
<int_traits <T1>::precision> > result_type;
};
- template <>
template <typename T1, typename T2>
struct binary_traits <T1, T2, CONST_PRECISION, CONST_PRECISION>
{
@@ -417,7 +411,6 @@ namespace wi
<int_traits <T1>::precision> > result_type;
};
- template <>
template <typename T1, typename T2>
struct binary_traits <T1, T2, VAR_PRECISION, VAR_PRECISION>
{
@@ -881,7 +874,6 @@ generic_wide_int <storage>::dump () const
namespace wi
{
- template <>
template <typename storage>
struct int_traits < generic_wide_int <storage> >
: public wi::int_traits <storage>
@@ -960,7 +952,6 @@ inline wide_int_ref_storage <SE>::wide_int_ref_storage (const T &x,
namespace wi
{
- template <>
template <bool SE>
struct int_traits <wide_int_ref_storage <SE> >
{
@@ -1147,7 +1138,6 @@ public:
namespace wi
{
- template <>
template <int N>
struct int_traits < fixed_wide_int_storage <N> >
{
--
2.4.3
