[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