[PATCH] D128830: [Pipelines] Introduce DAE after ArgumentPromotion

Wed Aug 24 02:59:12 PDT 2022

Michael137 added subscribers: aprantl, dblaikie.
Michael137 added a comment.

In D128830#3745069 <https://reviews.llvm.org/D128830#3745069>, @psamolysov wrote:

> @Michael137 Thank you very much for the information!
>
> I'm not sure, but it looks like the introduced change of the `readnone` attribute to `readonly` might make impact on DWARF. Unfortunately, I have no idea should this changes in DWARF be fixed or just it is enough to actualize the test.
>
> I've reverted the patch to give our time to make the decision about DWARF generation.

Thanks!

@aprantl @dblaikie Looks like this needs to accommodate existing DWARF generation behaviour?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128830