[PATCH] D127727: [RISCV] Turn on SeparateConstOffsetFromGEPPass for RISC-V target and added optional modification strategy in it

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

In D127727#3582087 <https://reviews.llvm.org/D127727#3582087>, @eklepilkina wrote:

>> This should be two patches, one changing the pass and one enabling for RISC-V.
>
> As far as I want to turn pass with enabled strategy, should I wait approve and merge of the accepting strategy and only after this create the second review? Or create series of patches as mentionedin documentation https://llvm.org/docs/Phabricator.html#creating-a-patch-series?

You should create a series of patches.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127727