[PATCH] D50030: [ARM] Adjust AND immediates to make them cheaper to select.

[Sam Parker via Phabricator via llvm-commits]

samparker added a comment.

Hi Eli,

Would you mind adding a test case, perhaps similar to i16_cmpz, for the uxtb case? Also, I can see there's a test case for bic with -256, but I'm not sure about -2.. so could you also add that edge case as well?

thanks,



================
Comment at: test/CodeGen/ARM/Windows/vla.ll:17
 ; CHECK-SMALL-CODE:   adds [[R4:r[0-9]+]], #7
-; CHECK-SMALL-CODE:   bic [[R4]], [[R4]], #7
+; CHECK-SMALL-CODE:   bic [[R4]], [[R4]], #4
 ; CHECK-SMALL-CODE:   lsrs r4, [[R4]], #2
----------------
Just a bug in the existing test case?


Repository:
  rL LLVM

https://reviews.llvm.org/D50030