[PATCH] D109963: [AArch64] Split bitmask immediate of bitwise AND operation

JinGu Kang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 27 08:58:21 PDT 2021


jaykang10 added a comment.

In D109963#3024009 <https://reviews.llvm.org/D109963#3024009>, @dmgreen wrote:

> My point in moving them was that they could be simplified, especially if inlined. As far as I can tell the two calls to splitAndBitmaskImm are recalculating values that were already computed in isValidAndSplitBitmaskImm. Can we not just calculate the values once?

Let me update it.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109963/new/

https://reviews.llvm.org/D109963



More information about the llvm-commits mailing list