Re: [gomp4] Work around expand_GOACC_DIM_SIZE/expand_GOACC_DIM_POS ICEs

[Nathan Sidwell <nathan_sidwell at mentor dot com>]

On 08/12/15 08:46, Nathan Sidwell wrote:
> On 08/12/15 06:21, Thomas Schwinge wrote:
> > Hi!
>   OpenACC offloading code for Intel MIC doesn't make
> > a lot of sense (currently), but it is what's being done, and has caused a
> > lot of regressions in my testing, so I committed the following workaround
> > to gomp-4_0-branch in r226804.  A different approach would have been to
> > invent some machinery to not compile OpenACC offloading code on the Intel
> > MIC offloading path, but as the latter eventually is to support OpenACC
> > offloading, too (see
>
> The right solution is probably for the default validated_dims to set the size to
> unity, just like the host.  Thereby forcing any backend that wants to provide a
> larger size to override that hook.
>
> I.e. remove the #ifndef ACCEL_COMPILER from default_validate_dims in omp-low

for avoidance of doubt, I'll add that to the patch I have in progress.

nathan

--
Nathan Sidwell - Director, Sourcery Services - Mentor Embedded