[PATCH] D146423: [Analys] Added new context analys for BB/Inst

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 21 01:42:05 PDT 2023


nikic added a comment.

In D146423#4208625 <https://reviews.llvm.org/D146423#4208625>, @mkazantsev wrote:

> Actually it was I who proposed Dmitry to split it off into a separate analysis. The main advantage of this approach is that we can then create some static analysis tools that don't necessarily modify the IR to give you the same info.

I think it would still be preferable to start implementing this as a MetaRenamer improvement, and then extracting it into a separate analysis if we have an actual use case for it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146423



More information about the llvm-commits mailing list