[PATCH] D14380: [ARM] Combine BFIs together
James Molloy via llvm-commits
llvm-commits at lists.llvm.org
Wed Nov 11 05:11:31 PST 2015
jmolloy marked 2 inline comments as done.
================
Comment at: lib/Target/ARM/ARMISelLowering.cpp:9046
@@ +9045,3 @@
+//
+// Neither A nor B must be zero.
+static bool BitsProperlyConcatenate(const APInt &A, const APInt &B) {
----------------
I was treating LastActiveBitInA and FirstActiveBitInB as numbering starting from the MSB. This was confusing, so I've swapped to LSB-based numbering.
================
Comment at: lib/Target/ARM/ARMISelLowering.cpp:9067
@@ +9066,3 @@
+ APInt NewToMask, NewFromMask;
+ SDValue NewFrom = ParseBFI(V.getNode(), NewToMask, NewFromMask);
+ if (NewFrom != From) {
----------------
I thought about it again and there's nothing stopping us looking through a BFI with a different base.
Repository:
rL LLVM
http://reviews.llvm.org/D14380
More information about the llvm-commits
mailing list