diff options
author | Jakub Jelinek <jakub@redhat.com> | 2022-09-13 19:00:02 +0200 |
---|---|---|
committer | Jakub Jelinek <jakub@redhat.com> | 2022-09-13 19:00:02 +0200 |
commit | e11babbfac21163118b69dd25b468ade80dbe8de (patch) | |
tree | f76779016df0f9aff14bddbb9fb6797fa796117d /INSTALL | |
parent | 7d7e2149cdb6b0c37e01df7745cb551fbed317f6 (diff) | |
download | gcc-e11babbfac21163118b69dd25b468ade80dbe8de.zip gcc-e11babbfac21163118b69dd25b468ade80dbe8de.tar.gz gcc-e11babbfac21163118b69dd25b468ade80dbe8de.tar.bz2 |
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.
Diffstat (limited to 'INSTALL')
0 files changed, 0 insertions, 0 deletions