[PATCH] D98230: [LSR] Add reconciliation of unfoldable offsets

Yueh-Ting (eop) Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 24 19:14:21 PDT 2022


eopXD added a comment.

Hi Jonas,

I am trying to wrap my head around this patch. I would really appreciate
if you can tell me more on your approach. (or please correct me if I am
saying something stupid)

It sounds like you are trying to have an alternative `Formula`-s when two
or more `LSRUse` have mutual unfold-able offset. This sounds like what
`LSRInstance::GenerateConstantOffsets` is doing. Won't adding more
`Formula`-s to an `LSRUse` do the job (rather than overriding the offset)?


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

https://reviews.llvm.org/D98230



More information about the llvm-commits mailing list