[PATCH] D31113: [AArch64] Add new subtarget feature to fold LSL into address mode.

[Chad Rosier via Phabricator via llvm-commits]

mcrosier added inline comments.


================
Comment at: lib/Target/AArch64/AArch64ISelDAGToDAG.cpp:347
+    unsigned Val = C->getZExtValue();
+    return (Subtarget->hasLSLFast() && Val < 4);
+  }
----------------
junbuml wrote:
> I think "Subtarget->hasLSLFast()" need to be check earlier. Maybe in isWorthFolding() ?
+1


https://reviews.llvm.org/D31113