https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81752

Thomas Schwinge <tschwinge at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Target|nvptx                       |
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2017-08-08
          Component|libgomp                     |other
            Summary|num_gangs(65536) converted  |Sanity-check OMP clause
                   |to 0                        |arguments, command-line
                   |                            |flags, and environment
                   |                            |variables
     Ever confirmed|0                           |1

--- Comment #1 from Thomas Schwinge <tschwinge at gcc dot gnu.org> ---
We're often using "unsigned short" to store dimension clauses, and sometimes
"int" or "unsigned int".  I'd say that "unsigned short" is an OK limitation,
but we should add compile-time overflow checks whenever we evaluate constant
clause arguments' values, and compile-time type-size checks for non-constant
arguments (actually, the latter check would work for both cases?),
corresponding checks for "-fopenacc-dim" parsing, and run-time checks in
libgomp for the "GOMP_OPENACC_DIM" environment variable.


And then, the same issue no doubt applies to other OMP clause arguments,
command-line flags, and environment variables, too.  (Thus adjusting the
"Summary".)

Reply via email to