[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