[COMMITTED] Add cases for CFN_BUILT_IN_SIGNBIT[FL].

Message ID 20221014100316.568795-1-aldyh@redhat.com
State Repeat Merge
Headers
Series [COMMITTED] Add cases for CFN_BUILT_IN_SIGNBIT[FL]. |

Checks

Context Check Description
snail/gcc-patch-check warning Git am fail log

Commit Message

Aldy Hernandez Oct. 14, 2022, 10:03 a.m. UTC
  gcc/ChangeLog:

	* gimple-range-op.cc
	(gimple_range_op_handler::maybe_builtin_call): Add
	CFN_BUILT_IN_SIGNBIT[FL]* entries.
---
 gcc/gimple-range-op.cc | 2 ++
 1 file changed, 2 insertions(+)
  

Comments

Jakub Jelinek Oct. 14, 2022, 10:16 a.m. UTC | #1
On Fri, Oct 14, 2022 at 12:03:16PM +0200, Aldy Hernandez via Gcc-patches wrote:
> gcc/ChangeLog:
> 
> 	* gimple-range-op.cc
> 	(gimple_range_op_handler::maybe_builtin_call): Add
> 	CFN_BUILT_IN_SIGNBIT[FL]* entries.
> ---
>  gcc/gimple-range-op.cc | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/gcc/gimple-range-op.cc b/gcc/gimple-range-op.cc
> index bc4389eb2e1..9bdef3d45c0 100644
> --- a/gcc/gimple-range-op.cc
> +++ b/gcc/gimple-range-op.cc
> @@ -757,6 +757,8 @@ gimple_range_op_handler::maybe_builtin_call ()
>        break;
>  
>      case CFN_BUILT_IN_SIGNBIT:
> +    case CFN_BUILT_IN_SIGNBITF:
> +    case CFN_BUILT_IN_SIGNBITL:

    CASE_FLT_FN (CFN_BUILT_IN_SIGNBIT):
instead of the above 3?

	Jakub
  
Aldy Hernandez Oct. 14, 2022, 2:23 p.m. UTC | #2
Done.

Thanks.
Aldy

On Fri, Oct 14, 2022 at 12:17 PM Jakub Jelinek <jakub@redhat.com> wrote:
>
> On Fri, Oct 14, 2022 at 12:03:16PM +0200, Aldy Hernandez via Gcc-patches wrote:
> > gcc/ChangeLog:
> >
> >       * gimple-range-op.cc
> >       (gimple_range_op_handler::maybe_builtin_call): Add
> >       CFN_BUILT_IN_SIGNBIT[FL]* entries.
> > ---
> >  gcc/gimple-range-op.cc | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/gcc/gimple-range-op.cc b/gcc/gimple-range-op.cc
> > index bc4389eb2e1..9bdef3d45c0 100644
> > --- a/gcc/gimple-range-op.cc
> > +++ b/gcc/gimple-range-op.cc
> > @@ -757,6 +757,8 @@ gimple_range_op_handler::maybe_builtin_call ()
> >        break;
> >
> >      case CFN_BUILT_IN_SIGNBIT:
> > +    case CFN_BUILT_IN_SIGNBITF:
> > +    case CFN_BUILT_IN_SIGNBITL:
>
>     CASE_FLT_FN (CFN_BUILT_IN_SIGNBIT):
> instead of the above 3?
>
>         Jakub
>
  

Patch

diff --git a/gcc/gimple-range-op.cc b/gcc/gimple-range-op.cc
index bc4389eb2e1..9bdef3d45c0 100644
--- a/gcc/gimple-range-op.cc
+++ b/gcc/gimple-range-op.cc
@@ -757,6 +757,8 @@  gimple_range_op_handler::maybe_builtin_call ()
       break;
 
     case CFN_BUILT_IN_SIGNBIT:
+    case CFN_BUILT_IN_SIGNBITF:
+    case CFN_BUILT_IN_SIGNBITL:
       m_op1 = gimple_call_arg (call, 0);
       m_float = &op_cfn_signbit;
       m_valid = true;