diff options
author | Jakub Jelinek <jakub@redhat.com> | 2020-04-27 16:05:03 +0200 |
---|---|---|
committer | Jakub Jelinek <jakub@redhat.com> | 2020-04-27 16:10:24 +0200 |
commit | 26d76be7af6db75aaab662f4e93395f4ff8acb38 (patch) | |
tree | c00ee59fc9ba9b3a2acb841a5d3d8f0fd371055a /gcc/c-family/ChangeLog | |
parent | 6b6a77d25e83e071ba5d08648c5bd3953d8a0c03 (diff) | |
download | gcc-26d76be7af6db75aaab662f4e93395f4ff8acb38.zip gcc-26d76be7af6db75aaab662f4e93395f4ff8acb38.tar.gz gcc-26d76be7af6db75aaab662f4e93395f4ff8acb38.tar.bz2 |
c-family: Fix ICE on __builtin_speculation_safe_value () [PR94755]
When this builtin has no parameters, speculation_safe_value_resolve_call
returns BUILT_IN_NONE, but resolve_overloaded_builtin uselessly
dereferences the first param just to return error_mark_node immediately.
The following patch rearranges it so that we only read the first parameter
if fncode is not BUILT_IN_NONE.
2020-04-27 Jakub Jelinek <jakub@redhat.com>
PR c/94755
* c-common.c (resolve_overloaded_builtin): Return error_mark_node for
fncode == BUILT_IN_NONE before initialization of first_param.
* c-c++-common/pr94755.c: New test.
Diffstat (limited to 'gcc/c-family/ChangeLog')
-rw-r--r-- | gcc/c-family/ChangeLog | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/gcc/c-family/ChangeLog b/gcc/c-family/ChangeLog index 55e6eb8..973dc70 100644 --- a/gcc/c-family/ChangeLog +++ b/gcc/c-family/ChangeLog @@ -1,3 +1,9 @@ +2020-04-27 Jakub Jelinek <jakub@redhat.com> + + PR c/94755 + * c-common.c (resolve_overloaded_builtin): Return error_mark_node for + fncode == BUILT_IN_NONE before initialization of first_param. + 2020-04-23 Marek Polacek <polacek@redhat.com> PR c++/94733 |