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

Eric Christopher echristo at gmail.com
Fri Jul 10 09:44:55 PDT 2015


echristo added a comment.

One inline question, LGTM otherwise though.

-eric


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


http://reviews.llvm.org/D11079







More information about the llvm-commits mailing list