[PATCH] D11028: Make TargetLowering::getPointerTy() taking DataLayout as an argument

[Eric Christopher]

echristo added a comment.

One inline question...

-eric


================
Comment at: include/llvm/CodeGen/BasicTTIImpl.h:149
@@ -148,3 +148,3 @@
   bool isTypeLegal(Type *Ty) {
-    EVT VT = getTLI()->getValueType(Ty);
+    EVT VT = getTLI()->getValueType(TargetTransformInfoImplBase::DL, Ty);
     return getTLI()->isTypeLegal(VT);
----------------
My C++ knowledge may be lacking here, but why can't we use getDataLayout here?


http://reviews.llvm.org/D11028