[Bug target/91638] powerpc -mlong-double-NN (documentation) issues

2020-03-09 Thread wschmidt at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91638

Bill Schmidt  changed:

   What|Removed |Added

 Status|WAITING |RESOLVED
 Resolution|--- |FIXED
 CC||wschmidt at gcc dot gnu.org

--- Comment #8 from Bill Schmidt  ---
Work is complete.

[Bug target/91638] powerpc -mlong-double-NN (documentation) issues

2020-03-09 Thread cel at us dot ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91638

Carl Love  changed:

   What|Removed |Added

 CC||cel at us dot ibm.com

--- Comment #7 from Carl Love  ---
Patch approved by Segher with a few minor fixes to the patch.

Patch committed to mainline

commit 9439378f7a08cf9c8f524c9f3758a37d804ac106
Author: Carl Love 
Date:   Thu Mar 5 12:52:35 2020 -0600

rs6000: Fix -mlong-double documentation

gcc/ChangeLog

2020-03-09  Carl Love  

* config/rs6000/rs6000.opt: Update the description of the
command line option.

[Bug target/91638] powerpc -mlong-double-NN (documentation) issues

2020-03-06 Thread cel at us dot ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91638

--- Comment #6 from Carl Love  ---
Yea, I like that a bit better.  It is a bit shorter, mine was a bit verbose.

I updated the patch to print:

  -mlong-double-  Use -mlong-double-64 for 64-bit IEEE floating
  point format. Use -mlong-double-128 for 128-bit
  floating point format (either IEEE or IBM).

I will post the patch to the mailing list for review/acceptance on mainline.

[Bug target/91638] powerpc -mlong-double-NN (documentation) issues

2020-03-05 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91638

--- Comment #5 from Segher Boessenkool  ---
Yeah, not sure.  A user can just do `-mlong-double-128`, and will get whichever
128-bit format is the default.  That is the "old" way, when we only had the
ibm128 format, and it will be the new way too, when ibm128 has become a bad
dream and everything uses ieee128.

Maybe just something like

Use -mlong-double-128 for an 128-bit floating point format (either IEEE or
IBM)?

[Bug target/91638] powerpc -mlong-double-NN (documentation) issues

2020-03-05 Thread cel at us dot ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91638

--- Comment #4 from Carl Love  ---
Created attachment 47982
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=47982&action=edit
patch to update -mlong-double-NN description

Attached the proposed patch so I will not lose it.

[Bug target/91638] powerpc -mlong-double-NN (documentation) issues

2020-03-05 Thread cel at us dot ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91638

Carl Love  changed:

   What|Removed |Added

 CC||cel at us dot ibm.com

--- Comment #3 from Carl Love  ---
I have looked at what -mlong-double-64 and -mlong-double-128 do.  

The -mlong-double-64 tells GCC to use the 64-bit IEEE floating point format.

The -mlong-double-128 tells GCC to use 128-bit floating point format.  On
ppc64, there are actually two 128-bit floating point formats that are
supported.  So it looks to me like the user should also specify either
-mabi=ieeelongdouble to get the 128-bit IEEE floating point format or
-mabi=ibmlongdouble to get the IBM 128-bit floating point format.  

I created a patch that updates the output of the gcc --target-help  command to
print the following:

  -mlong-double-  Use -mlong-double-64 for 64-bit IEEE floating
  point format. Use -mlong-double-128 and
  -mabi=ieeelongdouble for 128-bit IEEE floating
  point format. Use -mlong-double-128 and
  -mabi=ibmlongdouble for 128-bit IBM floating
  point format.

Please let me know if this description is accurate and better.  Suggestions on
improvements are always welcome.

[Bug target/91638] powerpc -mlong-double-NN (documentation) issues

2020-02-06 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91638

--- Comment #2 from Segher Boessenkool  ---
That isn't documentation.

[Bug target/91638] powerpc -mlong-double-NN (documentation) issues

2020-01-29 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91638

Martin Liška  changed:

   What|Removed |Added

 Status|UNCONFIRMED |WAITING
   Last reconfirmed||2020-01-29
 CC||marxin at gcc dot gnu.org
 Ever confirmed|0   |1

--- Comment #1 from Martin Liška  ---
I see a documentation:

$ powerpc64le-suse-linux-gcc --help=target | grep long-do
  -mlong-double-[64,128]  Specify size of long double.