On 05/23/2016 11:09 PM, Jakub Jelinek wrote:
> On Mon, May 23, 2016 at 07:31:53PM -0700, Cesar Philippidis wrote:
>> @@ -12559,7 +12560,7 @@ c_finish_omp_clauses (tree clauses, enum c_omp_region_type ort)
>>  	  t = OMP_CLAUSE_DECL (c);
>>  	  if (TREE_CODE (t) == TREE_LIST)
>>  	    {
>> -	      if (handle_omp_array_sections (c, ort & C_ORT_OMP))
>> +	      if (handle_omp_array_sections (c, ort & (C_ORT_OMP | C_ORT_ACC)))
>>  		{
>>  		  remove = true;
>>  		  break;
> 
> You haven't touched the /c/ handle_omp_array_sections{,_1}.  As I said, I believe
> you can just drop the is_omp argument altogether (unlike C++), or, pass for
> consistency ort itself there as well.  But I bet the argument will be
> unused.

OK, I removed is_omp. I only had to guard one call to
handle_omp_array_sections from c_finish_omp_clauses because OpenACC
doesn't support array reductions.

Is this OK for trunk?

Cesar