[Bug analyzer/93543] [10 regression] bootstrap with clang fails in analyzer: reinterpret_cast from 'nullptr_t' to 'function *' is not allowed

2020-02-04 Thread gerald at pfeifer dot com 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93543

--- Comment #7 from Gerald Pfeifer  ---
Thank you - and I just received confirmation that also the new patch
addresses the original issue coming from a FreeBSD user with clang 9.0.1.

[Bug analyzer/93543] [10 regression] bootstrap with clang fails in analyzer: reinterpret_cast from 'nullptr_t' to 'function *' is not allowed

2020-02-04 Thread dmalcolm at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93543

David Malcolm  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #6 from David Malcolm  ---
(In reply to Sebastian Huber from comment #4)
> (In reply to David Malcolm from comment #3)
> > Thanks.  Does the patch in comment #1 fix it for you?
> 
> I tested the patch on FreeBSD 12.1 with LLVM 8.0.1 and it fixes the issue.

Thanks!  Should now be fixed in master as per comment #5.

[Bug analyzer/93543] [10 regression] bootstrap with clang fails in analyzer: reinterpret_cast from 'nullptr_t' to 'function *' is not allowed

2020-02-04 Thread cvs-commit at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93543

--- Comment #5 from CVS Commits  ---
The master branch has been updated by David Malcolm :

https://gcc.gnu.org/g:1dae549dccfec1edb0cb4e65feadc4722bb3bcc8

commit r10-6434-g1dae549dccfec1edb0cb4e65feadc4722bb3bcc8
Author: David Malcolm 
Date:   Tue Jan 28 16:31:01 2020 -0500

analyzer: fix build error with clang (PR 93543)

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=243681 reports a build
failure with clang 9.0.1:

gcc/analyzer/engine.cc:2971:13: error:
  reinterpret_cast from 'nullptr_t' to 'function *' is not allowed
  v.m_fun = reinterpret_cast (NULL);
^~~
engine.cc:2983:21: error:
  reinterpret_cast from 'nullptr_t' to 'function *' is not allowed
  return v.m_fun == reinterpret_cast (NULL);
^~~

The casts appears to be unnecessary; eliminate them.

gcc/analyzer/ChangeLog:
PR analyzer/93543
* engine.cc (pod_hash_traits::mark_empty):
Eliminate reinterpret_cast.
(pod_hash_traits::is_empty): Likewise.

[Bug analyzer/93543] [10 regression] bootstrap with clang fails in analyzer: reinterpret_cast from 'nullptr_t' to 'function *' is not allowed

2020-02-04 Thread sebastian.hu...@embedded-brains.de 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93543

--- Comment #4 from Sebastian Huber  ---
(In reply to David Malcolm from comment #3)
> Thanks.  Does the patch in comment #1 fix it for you?

I tested the patch on FreeBSD 12.1 with LLVM 8.0.1 and it fixes the issue.

[Bug analyzer/93543] [10 regression] bootstrap with clang fails in analyzer: reinterpret_cast from 'nullptr_t' to 'function *' is not allowed

2020-02-04 Thread dmalcolm at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93543

David Malcolm  changed:

   What|Removed |Added

Summary|[10 regression] bootstrap   |[10 regression] bootstrap
   |with clang 9.0.1 fails in   |with clang fails in
   |analyzer: reinterpret_cast  |analyzer: reinterpret_cast
   |from 'nullptr_t' to |from 'nullptr_t' to
   |'function *' is not allowed |'function *' is not allowed

--- Comment #3 from David Malcolm  ---
Thanks.  Does the patch in comment #1 fix it for you?