[PATCH] D24494: [LTO] Export DataLayoutStr instead of the whole module

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 12 22:59:07 PDT 2016


davide created this revision.
davide added reviewers: pcc, mehdi_amini.
davide added a subscriber: llvm-commits.
Herald added a subscriber: mehdi_amini.

Trying to address r281289 post-Commit review. Peter, Mehdi, is this more reasonable to you?

https://reviews.llvm.org/D24494

Files:
  include/llvm/LTO/LTO.h

Index: include/llvm/LTO/LTO.h
===================================================================
--- include/llvm/LTO/LTO.h
+++ include/llvm/LTO/LTO.h
@@ -227,6 +227,10 @@
                             symbol_iterator(Obj->symbol_end()));
   }

+  StringRef getDataLayoutStr() const {
+    return Obj->getModule().getDataLayoutStr();
+  }
+
   StringRef getSourceFileName() const {
     return Obj->getModule().getSourceFileName();
   }
@@ -234,10 +238,6 @@
   MemoryBufferRef getMemoryBufferRef() const {
     return Obj->getMemoryBufferRef();
   }
-
-  Module &getModule() const {
-    return Obj->getModule();
-  }
 };

 /// A ThinBackend defines what happens after the thin-link phase during ThinLTO.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D24494.71109.patch
Type: text/x-patch
Size: 705 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160913/0f0c532a/attachment.bin>


More information about the llvm-commits mailing list