[PATCH] D11037: Make TargetLowering::getShiftAmountTy() taking DataLayout as an argument

Eric Christopher echristo at gmail.com
Wed Jul 8 16:03:11 PDT 2015


echristo accepted this revision.
echristo added a comment.
This revision is now accepted and ready to land.

LGTM, wish we could get rid of that use...

-eric


================
Comment at: include/llvm/Target/TargetLowering.h:174
@@ -173,3 +173,3 @@
   }
-  virtual MVT getScalarShiftAmountTy(EVT LHSTy) const;
+  virtual MVT getScalarShiftAmountTy(EVT LHSTy, const DataLayout &DL) const;
 
----------------
Can you make sure the position of the DataLayout argument is consistent while you do this? (See above with getPointerTy).


http://reviews.llvm.org/D11037







More information about the llvm-commits mailing list