[PATCH] D150599: [ARC] Use backwards scavenging in frame index elimination

[Jay Foad via Phabricator via llvm-commits]

foad added a comment.

In D150599#4343332 <https://reviews.llvm.org/D150599#4343332>, @arsenm wrote:

> Since there are no required changes, why not just let this take the flip of the default? I think the option should be dropped ASAP and this just increases that diff

Because I think there is a faster route to removing `RegScavenger::forward` that does not require converting //all// targets to `supportsBackwardScavenger`. Only the targets that actually scavenge during the `replaceFrameIndices` loop need to be converted, and there are not too many of those.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150599