diff mbox

[1/2,AArch64] Add missing support for poly64x1_t

Message ID VI1PR0801MB2031F961763A805A570143F6FFFE0@VI1PR0801MB2031.eurprd08.prod.outlook.com
State New
Headers show

Commit Message

Tamar Christina Sept. 13, 2016, 12:39 p.m. UTC 
Hi all,

This fixes a bug in the name mangling which prevented mangling
of functions with return type void and arguments that require
any qualifiers.  e.g. void(unsigned int, unsigned int).  

Ran regression tests on aarch64-none-linux-gnu.

Ok for trunk?

Thanks,
Tamar

gcc/
2016-08-02  Tamar Christina  <tamar.christina@arm.com>

	* config/aarch64/aarch64-builtins.c 
	(aarch64_init_simd_builtins): Fixed mangler.

Comments

James Greenhalgh Sept. 13, 2016, 4:33 p.m. UTC | #1 
On Tue, Sep 13, 2016 at 01:39:39PM +0100, Tamar Christina wrote:
> Hi all,
> 
> This fixes a bug in the name mangling which prevented mangling
> of functions with return type void and arguments that require
> any qualifiers.  e.g. void(unsigned int, unsigned int).  
> 
> Ran regression tests on aarch64-none-linux-gnu.
> 
> Ok for trunk?

OK, thanks.

I've committed this on your behalf as r240117.

Thanks,
James
diff mbox

Patch

:100644 100644 9136910... 822d608... M	gcc/config/aarch64/aarch64-builtins.c

diff --git a/gcc/config/aarch64/aarch64-builtins.c b/gcc/config/aarch64/aarch64-builtins.c
index 9136910..822d608 100644
--- a/gcc/config/aarch64/aarch64-builtins.c
+++ b/gcc/config/aarch64/aarch64-builtins.c
@@ -762,16 +762,16 @@  aarch64_init_simd_builtins (void)
 
 	  if (qualifiers & qualifier_unsigned)
 	    {
-	      type_signature[arg_num] = 'u';
+	      type_signature[op_num] = 'u';
 	      print_type_signature_p = true;
 	    }
 	  else if (qualifiers & qualifier_poly)
 	    {
-	      type_signature[arg_num] = 'p';
+	      type_signature[op_num] = 'p';
 	      print_type_signature_p = true;
 	    }
 	  else
-	    type_signature[arg_num] = 's';
+	    type_signature[op_num] = 's';
 
 	  /* Skip an internal operand for vget_{low, high}.  */
 	  if (qualifiers & qualifier_internal)