Sandra Loosemore wrote:
This patch is the C equivalent of commit r15-6512-gcf94ba812ca496 for C++,
to improve the location information for individual items in an OpenMP
variable list.

LGTM – thanks for the patch, improving the diagnostic and reducing the differences between C and C++ for when there no good reason for it.


gcc/c/ChangeLog
        PR c/118579
        * c-parser.cc (c_parser_omp_variable_list): Capture location
        information when KIND is OMP_CLAUSE_ERROR.
        (c_parser_oacc_data_clause_deviceptr): Use the improved location
        for diagnostics, and remove the FIXME.
        (c_finish_omp_declare_variant): Likewise.
        (c_parser_omp_threadprivate): Likewise.

gcc/testsuite/ChangeLog
        PR c/118579
        * c-c++-common/gomp/pr118579.c: New testcase.

Tobias

Reply via email to