kmclaughlin marked 4 inline comments as done.
kmclaughlin added a comment.
Thanks for taking a look at this, @sdesmalen!
================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:7496
+ auto CIdx = dyn_cast<ConstantSDNode>(Idx128);
+ if (CIdx && (CIdx->getZExtValue() <= 3)) {
+ auto CI = DAG.getTargetConstant(CIdx->getZExtValue(), DL, MVT::i64);
----------------
sdesmalen wrote:
> nit: can you replace `auto` in these cases with SDValue? (which I think this
> is?)
Replaced other cases of auto here with SDValue or SDNode
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D74734/new/
https://reviews.llvm.org/D74734
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
