svn commit: r379320 - head/devel/binutils/files

Dimitry Andric dim at FreeBSD.org
Wed Feb 18 22:23:08 UTC 2015


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 at 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.  */


More information about the svn-ports-all mailing list