[PATCH] D11079: Remove DataLayout from TargetLoweringObjectFile, redirect to Module

Mehdi AMINI mehdi.amini at apple.com
Mon Jul 13 11:06:19 PDT 2015


joker.eph added inline comments.

================
Comment at: lib/CodeGen/AsmPrinter/AsmPrinter.cpp:140
@@ -139,3 +139,3 @@
 const DataLayout &AsmPrinter::getDataLayout() const {
-  return *TM.getDataLayout();
+  return MMI->getModule()->getDataLayout();
 }
----------------
echristo wrote:
> Is this going to have problems when you don't have a module (ala some of the other uses?).
They never call this method (if they do, they're doomed).


http://reviews.llvm.org/D11079







More information about the llvm-commits mailing list