public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/97436] New: [nvptx] -m32 support
@ 2020-10-15  7:57 vries at gcc dot gnu.org
  2020-10-15  9:08 ` [Bug target/97436] " tschwinge at gcc dot gnu.org
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: vries at gcc dot gnu.org @ 2020-10-15  7:57 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97436

            Bug ID: 97436
           Summary: [nvptx] -m32 support
           Product: gcc
           Version: 11.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: vries at gcc dot gnu.org
  Target Milestone: ---

The nvptx port has an -m32 switch:
...
m32
Target Report RejectNegative InverseMask(ABI64)
Generate code for a 32-bit ABI.
...
but the default is -m64:
...
#define TARGET_DEFAULT_TARGET_FLAGS MASK_ABI64
...
[ which perhaps should be related to the host being -m64 or -m32? ]

We're not building -m32/-m64 multilibs, so it seems we're not building the -m32
part by default.

I don't know if the -m32 path was ever tested, either in standalone or
offloading setting.

But since the switch is there, we should either build and test or deprecate it.

I'm not yet sure what would be a working setup in terms of card/drivers/OS.

At least for linux, at cuda 7.5 it's mentioned that: "Support for developing
and running 32-bit CUDA and OpenCL applications on 64-bit x86 Linux platforms
is deprecated".

And with cuda 8.0, I get:
...
$ ~/cuda/8.0/bin/nvcc ~/hello.cu -m32
nvcc warning : Compiling in the 32-bit mode when the host compiler targets x86
or x86_64 is no longer supported on Linux
...

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

* [Bug target/97436] [nvptx] -m32 support
  2020-10-15  7:57 [Bug target/97436] New: [nvptx] -m32 support vries at gcc dot gnu.org
@ 2020-10-15  9:08 ` tschwinge at gcc dot gnu.org
  2020-10-15 13:26 ` [Bug target/97436] [nvptx] Remove " cvs-commit at gcc dot gnu.org
  2020-10-15 13:34 ` vries at gcc dot gnu.org
  2 siblings, 0 replies; 4+ messages in thread
From: tschwinge at gcc dot gnu.org @ 2020-10-15  9:08 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97436

Thomas Schwinge <tschwinge at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |tschwinge at gcc dot gnu.org
   Last reconfirmed|                            |2020-10-15
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |NEW

--- Comment #1 from Thomas Schwinge <tschwinge at gcc dot gnu.org> ---
We originally had been asked to support OpenACC/nvptx offloading with both
64-bit x86_64 and 32-bit x86, but the idea of a 32-bit variant got scrapped
very early in the project.  I don't think we've ever tested that, and it
doesn't serve any practical use.  I suggest to remove.

See also PR65099.

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

* [Bug target/97436] [nvptx] Remove -m32 support
  2020-10-15  7:57 [Bug target/97436] New: [nvptx] -m32 support vries at gcc dot gnu.org
  2020-10-15  9:08 ` [Bug target/97436] " tschwinge at gcc dot gnu.org
@ 2020-10-15 13:26 ` cvs-commit at gcc dot gnu.org
  2020-10-15 13:34 ` vries at gcc dot gnu.org
  2 siblings, 0 replies; 4+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2020-10-15 13:26 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97436

--- Comment #2 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Tom de Vries <vries@gcc.gnu.org>:

https://gcc.gnu.org/g:34af17c0164f3138df094b144c7f74c2d1805444

commit r11-3953-g34af17c0164f3138df094b144c7f74c2d1805444
Author: Tom de Vries <tdevries@suse.de>
Date:   Thu Oct 15 11:37:43 2020 +0200

    [nvptx] Remove -m32

    The nvptx port has an -m32 option, but it's not clear whether this
    was ever build/tested/used.

    Don't expose to user anymore.  Tested on nvptx.

    gcc/ChangeLog:

    2020-10-15  Tom de Vries  <tdevries@suse.de>

            PR target/97436
            * config/nvptx/nvptx.opt (m32): Comment out.
            * doc/invoke.texi (NVPTX options): Remove -m32.

    gcc/testsuite/ChangeLog:

    2020-10-15  Tom de Vries  <tdevries@suse.de>

            PR target/97436
            * gcc.target/nvptx/atomic_fetch-3.c: Remove.

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

* [Bug target/97436] [nvptx] Remove -m32 support
  2020-10-15  7:57 [Bug target/97436] New: [nvptx] -m32 support vries at gcc dot gnu.org
  2020-10-15  9:08 ` [Bug target/97436] " tschwinge at gcc dot gnu.org
  2020-10-15 13:26 ` [Bug target/97436] [nvptx] Remove " cvs-commit at gcc dot gnu.org
@ 2020-10-15 13:34 ` vries at gcc dot gnu.org
  2 siblings, 0 replies; 4+ messages in thread
From: vries at gcc dot gnu.org @ 2020-10-15 13:34 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97436

Tom de Vries <vries at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED
   Target Milestone|---                         |11.0

--- Comment #3 from Tom de Vries <vries at gcc dot gnu.org> ---
Updated release notes, marking resolved-fixed.

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

end of thread, other threads:[~2020-10-15 13:34 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-15  7:57 [Bug target/97436] New: [nvptx] -m32 support vries at gcc dot gnu.org
2020-10-15  9:08 ` [Bug target/97436] " tschwinge at gcc dot gnu.org
2020-10-15 13:26 ` [Bug target/97436] [nvptx] Remove " cvs-commit at gcc dot gnu.org
2020-10-15 13:34 ` vries at gcc dot gnu.org

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).