Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 25 May 2017 16:15:19 +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-11@freebsd.org
Subject:   svn commit: r318882 - in stable/11/contrib/llvm: include/llvm/MC lib/MC lib/Target/ARM/MCTargetDesc
Message-ID:  <201705251615.v4PGFJZN098563@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: dim
Date: Thu May 25 16:15:19 2017
New Revision: 318882
URL: https://svnweb.freebsd.org/changeset/base/318882

Log:
  MFC r318655:
  
  Pull in r302416 from upstream llvm trunk (by Martin Storsjö):
  
    [ARM] Clear the constant pool cache on explicit .ltorg directives
  
    Multiple ldr pseudoinstructions with the same constant value will
    reuse the same constant pool entry. However, if the constant pool is
    explicitly flushed with a .ltorg directive, we should not try to
    reference constants in the previous pool any longer, since they may
    be out of range.
  
    This fixes assembling hand-written assembler source which repeatedly
    loads the same constant value, across a binary size larger than the
    pc-relative fixup range for ldr instructions (4096 bytes). Such
    assembler source already uses explicit .ltorg instructions to emit
    constant pools with regular intervals. However if we try to reuse
    constants emitted in earlier pools, they end up out of range.
  
    This makes the output of the testcase match what binutils gas does
    (prior to this patch, it would fail to assemble).
  
    Differential Revision: https://reviews.llvm.org/D32847
  
  This should fix "out of range pc-relative fixup value" errors, when
  compiling certain ARM inline assembly for www/webkit-gtk[23].
  
  Reported by:	mmel

Modified:
  stable/11/contrib/llvm/include/llvm/MC/ConstantPools.h
  stable/11/contrib/llvm/lib/MC/ConstantPools.cpp
  stable/11/contrib/llvm/lib/Target/ARM/MCTargetDesc/ARMTargetStreamer.cpp
Directory Properties:
  stable/11/   (props changed)

Modified: stable/11/contrib/llvm/include/llvm/MC/ConstantPools.h
==============================================================================
--- stable/11/contrib/llvm/include/llvm/MC/ConstantPools.h	Thu May 25 14:54:22 2017	(r318881)
+++ stable/11/contrib/llvm/include/llvm/MC/ConstantPools.h	Thu May 25 16:15:19 2017	(r318882)
@@ -60,6 +60,8 @@ public:
 
   // Return true if the constant pool is empty
   bool empty();
+
+  void clearCache();
 };
 
 class AssemblerConstantPools {
@@ -83,6 +85,7 @@ class AssemblerConstantPools {
 public:
   void emitAll(MCStreamer &Streamer);
   void emitForCurrentSection(MCStreamer &Streamer);
+  void clearCacheForCurrentSection(MCStreamer &Streamer);
   const MCExpr *addEntry(MCStreamer &Streamer, const MCExpr *Expr,
                          unsigned Size, SMLoc Loc);
 

Modified: stable/11/contrib/llvm/lib/MC/ConstantPools.cpp
==============================================================================
--- stable/11/contrib/llvm/lib/MC/ConstantPools.cpp	Thu May 25 14:54:22 2017	(r318881)
+++ stable/11/contrib/llvm/lib/MC/ConstantPools.cpp	Thu May 25 16:15:19 2017	(r318882)
@@ -54,6 +54,10 @@ const MCExpr *ConstantPool::addEntry(con
 
 bool ConstantPool::empty() { return Entries.empty(); }
 
+void ConstantPool::clearCache() {
+  CachedEntries.clear();
+}
+
 //
 // AssemblerConstantPools implementation
 //
@@ -95,6 +99,13 @@ void AssemblerConstantPools::emitForCurr
   }
 }
 
+void AssemblerConstantPools::clearCacheForCurrentSection(MCStreamer &Streamer) {
+  MCSection *Section = Streamer.getCurrentSectionOnly();
+  if (ConstantPool *CP = getConstantPool(Section)) {
+    CP->clearCache();
+  }
+}
+
 const MCExpr *AssemblerConstantPools::addEntry(MCStreamer &Streamer,
                                                const MCExpr *Expr,
                                                unsigned Size, SMLoc Loc) {

Modified: stable/11/contrib/llvm/lib/Target/ARM/MCTargetDesc/ARMTargetStreamer.cpp
==============================================================================
--- stable/11/contrib/llvm/lib/Target/ARM/MCTargetDesc/ARMTargetStreamer.cpp	Thu May 25 14:54:22 2017	(r318881)
+++ stable/11/contrib/llvm/lib/Target/ARM/MCTargetDesc/ARMTargetStreamer.cpp	Thu May 25 16:15:19 2017	(r318882)
@@ -33,6 +33,7 @@ const MCExpr *ARMTargetStreamer::addCons
 
 void ARMTargetStreamer::emitCurrentConstantPool() {
   ConstantPools->emitForCurrentSection(Streamer);
+  ConstantPools->clearCacheForCurrentSection(Streamer);
 }
 
 // finish() - write out any non-empty assembler constant pools.



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