diff options
author | Frederik Harwath <frederik@codesourcery.com> | 2020-04-21 12:36:14 +0200 |
---|---|---|
committer | Frederik Harwath <frederik@codesourcery.com> | 2020-04-21 15:12:05 +0200 |
commit | 15256c8a8ac6573d250506c40dbe13082186c2aa (patch) | |
tree | 08bb1a7a75b85246792c18c03475348c29f9bc40 /libgcc | |
parent | ac1a0a388cd9529b4ba8b10d8ccd8099a2a95b23 (diff) | |
download | gcc-15256c8a8ac6573d250506c40dbe13082186c2aa.zip gcc-15256c8a8ac6573d250506c40dbe13082186c2aa.tar.gz gcc-15256c8a8ac6573d250506c40dbe13082186c2aa.tar.bz2 |
Remove fishy self-assignment in omp-low.c [PR94629]
The PR noticed that omp-low.c contains a self-assignment in the
function new_omp_context:
if (outer_ctx) {
...
ctx->outer_reduction_clauses = ctx->outer_reduction_clauses;
This is obviously useless. The original intention might have been
to copy the field from the outer_ctx to ctx. Since this is done
(properly) in the only function where this field is actually used
(in function scan_omp_for) and the field is being initialized to zero
during the struct allocation, there is no need to attempt to do
anything to this field in new_omp_context. Thus this commit
removes any assignment to the field from new_omp_context.
2020-04-21 Frederik Harwath <frederik@codesourcery.com>
PR other/94629
* gcc/omp-low.c (new_omp_context): Remove assignments to
ctx->outer_reduction_clauses and ctx->local_reduction_clauses.
Reviewed-by: Thomas Schwinge <thomas@codesourcery.com>
Diffstat (limited to 'libgcc')
0 files changed, 0 insertions, 0 deletions