public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [nvptx] some cleanups
@ 2015-10-28 18:36 Nathan Sidwell
  0 siblings, 0 replies; only message in thread
From: Nathan Sidwell @ 2015-10-28 18:36 UTC (permalink / raw)
  To: GCC Patches

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

I've committed this to trunk,  cleaning up a few bits and bobs I noticed 
comparing with the gomp4 branch.

nathan

[-- Attachment #2: trunk-nvptx-clean.patch --]
[-- Type: text/x-patch, Size: 2194 bytes --]

2015-10-28  Nathan Sidwell  <nathan@codesourcery.com>

	* config/nvptx/nvptx.c (nvptx_print_operand): Remove 'd' case.
	(struct parallel): Update comment.
	(nvptx_reorg): Likewise.
	(nvptx_neuter): Cleanup whitespace.

Index: config/nvptx/nvptx.c
===================================================================
--- config/nvptx/nvptx.c	(revision 229488)
+++ config/nvptx/nvptx.c	(working copy)
@@ -1969,7 +1969,6 @@ nvptx_print_operand_address (FILE *file,
 
    A -- print an address space identifier for a MEM
    c -- print an opcode suffix for a comparison operator, including a type code
-   d -- print a CONST_INT as a vector dimension (x, y, or z)
    f -- print a full reg even for something that must always be split
    S -- print a shuffle kind specified by CONST_INT
    t -- print a type opcode suffix, promoting QImode to 32 bits
@@ -2013,18 +2012,6 @@ nvptx_print_operand (FILE *file, rtx x,
       }
       break;
 
-    case 'd':
-      gcc_assert (x_code == CONST_INT);
-      if (INTVAL (x) == 0)
-	fputs (".x", file);
-      else if (INTVAL (x) == 1)
-	fputs (".y", file);
-      else if (INTVAL (x) == 2)
-	fputs (".z", file);
-      else
-	gcc_unreachable ();
-      break;
-
     case 't':
       op_mode = nvptx_underlying_object_mode (x);
       fprintf (file, "%s", nvptx_ptx_type_from_mode (op_mode, true));
@@ -2294,9 +2281,8 @@ nvptx_reorg_subreg (void)
     }
 }
 
-/* Loop structure of the function. The entire function is described as
-   a NULL loop.  We should be able to extend this to represent
-   superblocks.  */
+/* Loop structure of the function.  The entire function is described as
+   a NULL loop.  */
 
 struct parallel
 {
@@ -3028,7 +3014,6 @@ nvptx_neuter_pars (parallel *par, unsign
 }
 
 /* PTX-specific reorganization
-   - Scan and release reduction buffers
    - Split blocks at fork and join instructions
    - Compute live registers
    - Mark now-unused registers, so function begin doesn't declare
@@ -3214,11 +3199,11 @@ nvptx_record_offload_symbol (tree decl)
 	    
 	    fprintf (asm_out_file, ", %#x", size);
 	  }
-	
+
 	fprintf (asm_out_file, "\n");
       }
       break;
-  
+
     default:
       gcc_unreachable ();
     }

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2015-10-28 18:33 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-10-28 18:36 [nvptx] some cleanups Nathan Sidwell

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