Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 18 Feb 2015 22:23:07 +0000 (UTC)
From:      Dimitry Andric <dim@FreeBSD.org>
To:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   svn commit: r379320 - head/devel/binutils/files
Message-ID:  <201502182223.t1IMN7wX042780@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: dim (src committer)
Date: Wed Feb 18 22:23:06 2015
New Revision: 379320
URL: https://svnweb.freebsd.org/changeset/ports/379320
QAT: https://qat.redports.org/buildarchive/r379320/

Log:
  Fix undefined behavior in gas/contrib/tc-arm.c's rotate_left() macro,
  similar to https://svnweb.freebsd.org/base?view=revision&revision=274856
  
  This fixes possible "invalid constant after fixup" errors from gas, when
  it has been compiled with high levels of optimization.
  
  Obtained from:	binutils commit d840c081f8082e8b9e63fead5306643975a97bb3
  PR:		197806
  Approved by:	maintainer (bapt)

Added:
  head/devel/binutils/files/patch-gas_config_tc-arm.c   (contents, props changed)

Added: head/devel/binutils/files/patch-gas_config_tc-arm.c
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ head/devel/binutils/files/patch-gas_config_tc-arm.c	Wed Feb 18 22:23:06 2015	(r379320)
@@ -0,0 +1,17 @@
+From d840c081f8082e8b9e63fead5306643975a97bb3 Mon Sep 17 00:00:00 2001
+From: Richard Earnshaw <Richard.Earnshaw@arm.com>
+Date: Thu, 20 Nov 2014 17:02:47 +0000
+Subject: [PATCH] * config/tc-arm.c (rotate_left): Avoid undefined behaviour
+ when N = 0.
+
+--- gas/config/tc-arm.c.orig	2014-10-14 09:32:03.000000000 +0200
++++ gas/config/tc-arm.c	2015-02-18 21:13:40.720573000 +0100
+@@ -7251,7 +7251,7 @@
+ 
+ /* Functions for operand encoding.  ARM, then Thumb.  */
+ 
+-#define rotate_left(v, n) (v << n | v >> (32 - n))
++#define rotate_left(v, n) (v << (n & 31) | v >> ((32 - n) & 31))
+ 
+ /* If VAL can be encoded in the immediate field of an ARM instruction,
+    return the encoded form.  Otherwise, return FAIL.  */



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201502182223.t1IMN7wX042780>