[PATCH] D11000: Redirect DataLayout from TargetMachine to Module in SelectionDAG

[Eric Christopher]

echristo accepted this revision.

This revision is now accepted and ready to land.

One inline comment otherwise LGTM.

-eric


================
Comment at: lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp:6720
@@ -6720,2 +6719,3 @@
   SmallVector<uint64_t, 4> Offsets;
+  auto &DL = CLI.getDAG().getDataLayout();
   ComputeValueVTs(*this, CLI.RetTy, RetTys, &Offsets);
----------------
You can access the DAG directly here (as the code below shows). I don't mind an accessor, but cleaning that up should be a separate patch :)


Repository:
  rL LLVM

http://reviews.llvm.org/D11000