Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 3 Feb 2012 23:32:23 +0000 (UTC)
From:      Dimitry Andric <dim@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-9@freebsd.org
Subject:   svn commit: r230969 - stable/9/contrib/llvm/lib/Target/X86
Message-ID:  <201202032332.q13NWN3J009478@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: dim
Date: Fri Feb  3 23:32:22 2012
New Revision: 230969
URL: http://svn.freebsd.org/changeset/base/230969

Log:
  MFC r230393:
  
  Pull in r148240 from upstream llvm trunk:
  
    Make sure the non-SSE lowering for fences correctly clobbers EFLAGS.
    PR11768.
  
  In particular, this fixes segfaults during the build of devel/icu on
  i386.  The __sync_synchronize() builtin used for implementing icu's
  internal barrier could lead to incorrect behaviour.

Modified:
  stable/9/contrib/llvm/lib/Target/X86/X86InstrCompiler.td
Directory Properties:
  stable/9/contrib/llvm/   (props changed)

Modified: stable/9/contrib/llvm/lib/Target/X86/X86InstrCompiler.td
==============================================================================
--- stable/9/contrib/llvm/lib/Target/X86/X86InstrCompiler.td	Fri Feb  3 23:27:07 2012	(r230968)
+++ stable/9/contrib/llvm/lib/Target/X86/X86InstrCompiler.td	Fri Feb  3 23:32:22 2012	(r230969)
@@ -533,7 +533,7 @@ def ATOMSWAP6432 : I<0, Pseudo, (outs GR
 // Memory barriers
 
 // TODO: Get this to fold the constant into the instruction.
-let isCodeGenOnly = 1 in
+let isCodeGenOnly = 1, Defs = [EFLAGS] in
 def OR32mrLocked  : I<0x09, MRMDestMem, (outs), (ins i32mem:$dst, GR32:$zero),
                       "lock\n\t"
                       "or{l}\t{$zero, $dst|$dst, $zero}",



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