]> git.ipfire.org Git - thirdparty/gcc.git/commit - libgomp/env.c
libgomp: Appease some static analyzers [PR106906]
authorJakub Jelinek <jakub@redhat.com>
Tue, 13 Sep 2022 17:00:02 +0000 (19:00 +0200)
committerJakub Jelinek <jakub@redhat.com>
Tue, 13 Sep 2022 17:00:02 +0000 (19:00 +0200)
commite11babbfac21163118b69dd25b468ade80dbe8de
treef76779016df0f9aff14bddbb9fb6797fa796117d
parent7d7e2149cdb6b0c37e01df7745cb551fbed317f6
libgomp: Appease some static analyzers [PR106906]

While icv_addr[1] = false; assignments where icv_addr has void *
element type is correct and matches how it is used (in those cases
the void * pointer is then cast to bool and used that way), there is no
reason not to add explicit (void *) casts there which are there already
for (void *) true.  And, there is in fact even no point in actually
doing those stores at all because we set that pointer to NULL a few
lines earlier.  So, this patch adds the explicit casts and then
comments those out to show intent.

2022-09-13  Jakub Jelinek  <jakub@redhat.com>

PR libgomp/106906
* env.c (get_icv_member_addr): Cast false to void * before assigning
it to icv_addr[1], and comment the whole assignment out.
libgomp/env.c