On 11/07/2015 04:30 AM, Thomas Schwinge wrote:
> Hi!
> 
> On Fri, 6 Nov 2015 15:31:23 -0800, Cesar Philippidis <ce...@codesourcery.com> 
> wrote:
>> I've applied this patch to gomp-4_0-branch which backports most of my
>> front end changes from trunk.
> 
>> --- a/gcc/cp/pt.c
>> +++ b/gcc/cp/pt.c
>> @@ -14398,7 +14398,6 @@ tsubst_omp_clauses (tree clauses, bool declare_simd, 
>> bool allow_fields,
>>      case OMP_CLAUSE_NUM_GANGS:
>>      case OMP_CLAUSE_NUM_WORKERS:
>>      case OMP_CLAUSE_VECTOR_LENGTH:
>> -    case OMP_CLAUSE_GANG:
>>      case OMP_CLAUSE_WORKER:
>>      case OMP_CLAUSE_VECTOR:
>>      case OMP_CLAUSE_ASYNC:
>> @@ -14427,7 +14426,7 @@ tsubst_omp_clauses (tree clauses, bool declare_simd, 
>> bool allow_fields,
>>          = tsubst_omp_clause_decl (OMP_CLAUSE_DECL (oc), args, complain,
>>                                    in_decl);
>>        break;
>> -    case OMP_CLAUSE_LINEAR:
>> +    case OMP_CLAUSE_GANG:
>>      case OMP_CLAUSE_ALIGNED:
>>        OMP_CLAUSE_DECL (nc)
>>          = tsubst_omp_clause_decl (OMP_CLAUSE_DECL (oc), args, complain,
> 
> This -- unintentional, I suppose ;-) -- removal of OMP_CLAUSE_LINEAR
> caused a lot of regressions; committed to gomp-4_0-branch in r229928:

Thank you. I had two versions of this patch and I committed the wrong
one. That was the only change though.

Cesar

Reply via email to