From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 22609 invoked by alias); 1 Jun 2012 08:54:56 -0000 Received: (qmail 22600 invoked by uid 22791); 1 Jun 2012 08:54:55 -0000 X-SWARE-Spam-Status: No, hits=-0.9 required=5.0 tests=AWL,BAYES_00,KHOP_RCVD_UNTRUST,MSGID_MULTIPLE_AT,RCVD_IN_DNSWL_LOW,RCVD_IN_HOSTKARMA_NO X-Spam-Check-By: sourceware.org Received: from service87.mimecast.com (HELO service87.mimecast.com) (91.220.42.44) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Fri, 01 Jun 2012 08:54:42 +0000 Received: from cam-owa2.Emea.Arm.com (fw-tnat.cambridge.arm.com [217.140.96.21]) by service87.mimecast.com; Fri, 01 Jun 2012 09:54:10 +0100 Received: from e102132 ([10.1.255.212]) by cam-owa2.Emea.Arm.com with Microsoft SMTPSVC(6.0.3790.3959); Fri, 1 Jun 2012 09:54:19 +0100 From: "Sofiane Naci" To: Subject: [AArch64] Use snprintf instead sprintf Date: Fri, 01 Jun 2012 08:54:00 -0000 Message-ID: <000b01cd3fd4$11a512e0$34ef38a0$@naci@arm.com> MIME-Version: 1.0 X-MC-Unique: 112060109541042801 Content-Type: multipart/mixed; boundary="----=_NextPart_000_000C_01CD3FDC.73697AE0" X-IsSubscribed: yes Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org X-SW-Source: 2012-06/txt/msg00020.txt.bz2 This is a multi-part message in MIME format. ------=_NextPart_000_000C_01CD3FDC.73697AE0 Content-Type: text/plain; charset=WINDOWS-1252 Content-Transfer-Encoding: quoted-printable Content-length: 680 Hi, This patch replaces instances of sprintf with snprintf with sizeof(..) in the AArch64 port. It also fixes layout issues in the code it touches. Thanks Sofiane ----- ChangeLog 2012-06-01 Sofiane Naci [AArch64] Replace sprintf with snprintf. =20=09 * config/aarch64/aarch64.c (aarch64_elf_asm_constructor): Replace sprintf with snprintf. (aarch64_elf_asm_destructor): Likewise. (aarch64_output_casesi): Likewise. (aarch64_output_asm_insn): Likewise. * config/aarch64/aarch64-builtins.c (init_aarch64_simd_builtins): Likewise. * config/aarch64/aarch64-simd.md (*aarch64_simd_mov): Replace sprintf with snprintf, and fix code layout.= ------=_NextPart_000_000C_01CD3FDC.73697AE0 Content-Transfer-Encoding: base64 Content-Type: application/octet-stream; name="aarch64-snprintf.patch" Content-Disposition: attachment; filename="aarch64-snprintf.patch" Content-length: 4994 ZGlmZiAtLWdpdCBhL2djYy9jb25maWcvYWFyY2g2NC9hYXJjaDY0LWJ1aWx0 aW5zLmMgYi9nY2MvY29uZmlnL2FhcmNoNjQvYWFyY2g2NC1idWlsdGlucy5j CmluZGV4IDI5N2NhMTguLjE4NmI1ODkgMTAwNjQ0Ci0tLSBhL2djYy9jb25m aWcvYWFyY2g2NC9hYXJjaDY0LWJ1aWx0aW5zLmMKKysrIGIvZ2NjL2NvbmZp Zy9hYXJjaDY0L2FhcmNoNjQtYnVpbHRpbnMuYwpAQCAtMTAxNyw4ICsxMDE3 LDggQEAgaW5pdF9hYXJjaDY0X3NpbWRfYnVpbHRpbnMgKHZvaWQpCiAKIAkg IGdjY19hc3NlcnQgKGZ0eXBlICE9IE5VTEwpOwogCi0JICBzcHJpbnRmIChu YW1lYnVmLCAiX19idWlsdGluX2FhcmNoNjRfJXMlcyIsIGQtPm5hbWUsCi0J CSAgIG1vZGVuYW1lc1tqXSk7CisJICBzbnByaW50ZiAobmFtZWJ1Ziwgc2l6 ZW9mIChuYW1lYnVmKSwgIl9fYnVpbHRpbl9hYXJjaDY0XyVzJXMiLAorCQkg ICAgZC0+bmFtZSwgbW9kZW5hbWVzW2pdKTsKIAogCSAgYWRkX2J1aWx0aW5f ZnVuY3Rpb24gKG5hbWVidWYsIGZ0eXBlLCBmY29kZSsrLCBCVUlMVF9JTl9N RCwgTlVMTCwKIAkJCQlOVUxMX1RSRUUpOwpkaWZmIC0tZ2l0IGEvZ2NjL2Nv bmZpZy9hYXJjaDY0L2FhcmNoNjQtc2ltZC5tZCBiL2djYy9jb25maWcvYWFy Y2g2NC9hYXJjaDY0LXNpbWQubWQKaW5kZXggODVjOGMyYi4uZGM3OTVhOSAx MDA2NDQKLS0tIGEvZ2NjL2NvbmZpZy9hYXJjaDY0L2FhcmNoNjQtc2ltZC5t ZAorKysgYi9nY2MvY29uZmlnL2FhcmNoNjQvYWFyY2g2NC1zaW1kLm1kCkBA IC00MDcsMTQgKzQwNywxNCBAQAogCWlmIChtdm4gPT0gMCkKIAkgIHsKIAkg ICAgaWYgKHdpZHRoYyAhPSAnZCcpCi0JICAgICAgc3ByaW50ZiAodGVtcGws Im1vdmlcdCUlMC4lZCVjLCAlJTEsIGxzbCAlZCAiLCg2NC93aWR0aCksCi0J CQkJCQkJd2lkdGhjLCBzaGlmdCk7CisJICAgICAgc25wcmludGYgKHRlbXBs LCBzaXplb2YgKHRlbXBsKSwgIm1vdmlcdCUlMC4lZCVjLCAlJTEsIGxzbCAl ZCAiLAorCQkJNjQgLyB3aWR0aCwgd2lkdGhjLCBzaGlmdCk7CiAJICAgIGVs c2UKLQkgICAgICBzcHJpbnRmICh0ZW1wbCwibW92aVx0JSVkMCwgJSUxIik7 CisJICAgICAgc25wcmludGYgKHRlbXBsLCBzaXplb2YgKHRlbXBsKSwgIm1v dmlcdCUlZDAsICUlMSIpOwogCSAgfQogCWVsc2UKLQkgIHNwcmludGYgKHRl bXBsLCJtdm5pXHQlJTAuJWQlYywgJSUxLCBsc2wgJWQiLCg2NC93aWR0aCks Ci0JCQkJCQkJd2lkdGhjLCBzaGlmdCk7CisJICBzbnByaW50ZiAodGVtcGws IHNpemVvZiAodGVtcGwpLCAibXZuaVx0JSUwLiVkJWMsICUlMSwgbHNsICVk IiwKKwkJICAgIDY0IC8gd2lkdGgsIHdpZHRoYywgc2hpZnQpOwogCXJldHVy biB0ZW1wbDsKICAgICAgICB9CiAgICAgIGRlZmF1bHQ6IGdjY191bnJlYWNo YWJsZSAoKTsKQEAgLTQ1NSwxMSArNDU1LDExIEBACiAJCQkJCQkgJm12biwg JnNoaWZ0KTsKIAlnY2NfYXNzZXJ0IChpc192YWxpZCAhPSAwKTsKIAlpZiAo bXZuID09IDApCi0JICBzcHJpbnRmICh0ZW1wbCwibW92aVx0JSUwLiVkJWMs ICUlMSwgbHNsICVkICIsICgxMjgvd2lkdGgpLAotCQkJCQkJCSB3aWR0aGMs IHNoaWZ0KTsKKwkgIHNucHJpbnRmICh0ZW1wbCwgc2l6ZW9mICh0ZW1wbCks ICJtb3ZpXHQlJTAuJWQlYywgJSUxLCBsc2wgJWQgIiwKKwkJICAgIDEyOCAv IHdpZHRoLCB3aWR0aGMsIHNoaWZ0KTsKIAllbHNlCi0JICBzcHJpbnRmICh0 ZW1wbCwibXZuaVx0JSUwLiVkJWMsICUlMSwgbHNsICVkIiwgKDEyOC93aWR0 aCksCi0JCQkJCQkJd2lkdGhjLCBzaGlmdCk7CisJICBzbnByaW50ZiAodGVt cGwsIHNpemVvZiAodGVtcGwpLCAibXZuaVx0JSUwLiVkJWMsICUlMSwgbHNs ICVkIiwKKwkJICAgIDEyOCAvIHdpZHRoLCB3aWR0aGMsIHNoaWZ0KTsKIAly ZXR1cm4gdGVtcGw7CiAgICAgICAgfQogICAgICBkZWZhdWx0OiBnY2NfdW5y ZWFjaGFibGUgKCk7CmRpZmYgLS1naXQgYS9nY2MvY29uZmlnL2FhcmNoNjQv YWFyY2g2NC5jIGIvZ2NjL2NvbmZpZy9hYXJjaDY0L2FhcmNoNjQuYwppbmRl eCAzNGQwNDc1Li45MzNkN2Q2IDEwMDY0NAotLS0gYS9nY2MvY29uZmlnL2Fh cmNoNjQvYWFyY2g2NC5jCisrKyBiL2djYy9jb25maWcvYWFyY2g2NC9hYXJj aDY0LmMKQEAgLTM3NjgsNyArMzc2OCw3IEBAIGFhcmNoNjRfZWxmX2FzbV9j b25zdHJ1Y3RvciAocnR4IHN5bWJvbCwgaW50IHByaW9yaXR5KQogICAgIHsK ICAgICAgIHNlY3Rpb24gKnM7CiAgICAgICBjaGFyIGJ1ZlsxOF07Ci0gICAg ICBzcHJpbnRmIChidWYsICIuaW5pdF9hcnJheS4lLjV1IiwgcHJpb3JpdHkp OworICAgICAgc25wcmludGYgKGJ1Ziwgc2l6ZW9mIChidWYpLCAiLmluaXRf YXJyYXkuJS41dSIsIHByaW9yaXR5KTsKICAgICAgIHMgPSBnZXRfc2VjdGlv biAoYnVmLCBTRUNUSU9OX1dSSVRFLCBOVUxMKTsKICAgICAgIHN3aXRjaF90 b19zZWN0aW9uIChzKTsKICAgICAgIGFzc2VtYmxlX2FsaWduIChQT0lOVEVS X1NJWkUpOwpAQCAtMzc4Nyw3ICszNzg3LDcgQEAgYWFyY2g2NF9lbGZfYXNt X2Rlc3RydWN0b3IgKHJ0eCBzeW1ib2wsIGludCBwcmlvcml0eSkKICAgICB7 CiAgICAgICBzZWN0aW9uICpzOwogICAgICAgY2hhciBidWZbMThdOwotICAg ICAgc3ByaW50ZiAoYnVmLCAiLmZpbmlfYXJyYXkuJS41dSIsIHByaW9yaXR5 KTsKKyAgICAgIHNucHJpbnRmIChidWYsIHNpemVvZiAoYnVmKSwgIi5maW5p X2FycmF5LiUuNXUiLCBwcmlvcml0eSk7CiAgICAgICBzID0gZ2V0X3NlY3Rp b24gKGJ1ZiwgU0VDVElPTl9XUklURSwgTlVMTCk7CiAgICAgICBzd2l0Y2hf dG9fc2VjdGlvbiAocyk7CiAgICAgICBhc3NlbWJsZV9hbGlnbiAoUE9JTlRF Ul9TSVpFKTsKQEAgLTM4MzYsNyArMzgzNiw4IEBAIGFhcmNoNjRfb3V0cHV0 X2Nhc2VzaSAocnR4ICpvcGVyYW5kcykKICAgLyogTmVlZCB0byBpbXBsZW1l bnQgdGFibGUgc2l6ZSByZWR1Y3Rpb24sIGJ5IGNoYW5pbmcgdGhlIGNvZGUg YmVsb3cuICAqLwogICBvdXRwdXRfYXNtX2luc24gKHBhdHRlcm5zW2luZGV4 XVswXSwgb3BlcmFuZHMpOwogICBBU01fR0VORVJBVEVfSU5URVJOQUxfTEFC RUwgKGxhYmVsLCAiTHJ0eCIsIENPREVfTEFCRUxfTlVNQkVSIChvcGVyYW5k c1syXSkpOwotICBzcHJpbnRmIChidWYsICJhZHJcdCUlNCwgJXMiLCB0YXJn ZXRtLnN0cmlwX25hbWVfZW5jb2RpbmcgKGxhYmVsKSk7CisgIHNucHJpbnRm IChidWYsIHNpemVvZiAoYnVmKSwKKwkgICAgImFkclx0JSU0LCAlcyIsIHRh cmdldG0uc3RyaXBfbmFtZV9lbmNvZGluZyAobGFiZWwpKTsKICAgb3V0cHV0 X2FzbV9pbnNuIChidWYsIG9wZXJhbmRzKTsKICAgb3V0cHV0X2FzbV9pbnNu IChwYXR0ZXJuc1tpbmRleF1bMV0sIG9wZXJhbmRzKTsKICAgb3V0cHV0X2Fz bV9pbnNuICgiYnJcdCUzIiwgb3BlcmFuZHMpOwpAQCAtNTg5MSw3ICs1ODky LDcgQEAgYWFyY2g2NF9vdXRwdXRfYXNtX2luc24gKGVtaXRfZiBlbWl0LCBp bnQgbGFiZWwsIHJ0eCAqb3BlcmFuZHMsCiAgIGNoYXIgYnVmZmVyWzI1Nl07 CiAKICAgdmFfc3RhcnQgKGFwLCBwYXR0ZXJuKTsKLSAgdnNwcmludGYgKGJ1 ZmZlciwgcGF0dGVybiwgYXApOworICB2c25wcmludGYgKGJ1ZmZlciwgc2l6 ZW9mIChidWZmZXIpLCBwYXR0ZXJuLCBhcCk7CiAgIHZhX2VuZCAoYXApOwog ICBlbWl0IChsYWJlbCwgYnVmZmVyLCBvcGVyYW5kcyk7CiB9Cg== ------=_NextPart_000_000C_01CD3FDC.73697AE0--