public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [COMMITTED] Add cases for CFN_BUILT_IN_SIGNBIT[FL].
@ 2022-10-14 10:03 Aldy Hernandez
  2022-10-14 10:16 ` Jakub Jelinek
  0 siblings, 1 reply; 3+ messages in thread
From: Aldy Hernandez @ 2022-10-14 10:03 UTC (permalink / raw)
  To: GCC patches; +Cc: Andrew MacLeod, Aldy Hernandez

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:
       m_op1 = gimple_call_arg (call, 0);
       m_float = &op_cfn_signbit;
       m_valid = true;
-- 
2.37.3


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [COMMITTED] Add cases for CFN_BUILT_IN_SIGNBIT[FL].
  2022-10-14 10:03 [COMMITTED] Add cases for CFN_BUILT_IN_SIGNBIT[FL] Aldy Hernandez
@ 2022-10-14 10:16 ` Jakub Jelinek
  2022-10-14 14:23   ` Aldy Hernandez
  0 siblings, 1 reply; 3+ messages in thread
From: Jakub Jelinek @ 2022-10-14 10:16 UTC (permalink / raw)
  To: Aldy Hernandez; +Cc: GCC patches

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


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [COMMITTED] Add cases for CFN_BUILT_IN_SIGNBIT[FL].
  2022-10-14 10:16 ` Jakub Jelinek
@ 2022-10-14 14:23   ` Aldy Hernandez
  0 siblings, 0 replies; 3+ messages in thread
From: Aldy Hernandez @ 2022-10-14 14:23 UTC (permalink / raw)
  To: Jakub Jelinek; +Cc: GCC patches

[-- Attachment #1: Type: text/plain, Size: 888 bytes --]

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
>

[-- Attachment #2: 0003-Replace-CFN_BUILTIN_SIGNBIT-cases-with-CASE_FLT_FN.patch --]
[-- Type: text/x-patch, Size: 955 bytes --]

From f44a0f5530012e2a0108e9034c7d63adcd21d7ff Mon Sep 17 00:00:00 2001
From: Aldy Hernandez <aldyh@redhat.com>
Date: Fri, 14 Oct 2022 15:02:06 +0200
Subject: [PATCH] Replace CFN_BUILTIN_SIGNBIT* cases with CASE_FLT_FN.

gcc/ChangeLog:

	* gimple-range-op.cc
	(gimple_range_op_handler::maybe_builtin_call): Replace
	CFN_BUILTIN_SIGNBIT* cases with CASE_FLT_FN.
---
 gcc/gimple-range-op.cc | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/gcc/gimple-range-op.cc b/gcc/gimple-range-op.cc
index 9bdef3d45c0..527893f66af 100644
--- a/gcc/gimple-range-op.cc
+++ b/gcc/gimple-range-op.cc
@@ -756,9 +756,7 @@ gimple_range_op_handler::maybe_builtin_call ()
 	m_valid = false;
       break;
 
-    case CFN_BUILT_IN_SIGNBIT:
-    case CFN_BUILT_IN_SIGNBITF:
-    case CFN_BUILT_IN_SIGNBITL:
+    CASE_FLT_FN (CFN_BUILT_IN_SIGNBIT):
       m_op1 = gimple_call_arg (call, 0);
       m_float = &op_cfn_signbit;
       m_valid = true;
-- 
2.37.3


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-10-14 14:23 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-14 10:03 [COMMITTED] Add cases for CFN_BUILT_IN_SIGNBIT[FL] Aldy Hernandez
2022-10-14 10:16 ` Jakub Jelinek
2022-10-14 14:23   ` Aldy Hernandez

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).