diff mbox

[nvptx] some cleanups

Message ID 563114E7.40400@acm.org
State New
Headers show

Commit Message

Nathan Sidwell Oct. 28, 2015, 6:33 p.m. UTC
I've committed this to trunk,  cleaning up a few bits and bobs I noticed 
comparing with the gomp4 branch.

nathan
diff mbox

Patch

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 ();
     }