[PATCH] D144162: [AMDGPU] Replace LegacyDA with Uniformity Analysis in AnnotateUniformValues

Tue Feb 21 09:03:14 PST 2023

gandhi21299 added a comment.

In D144162#4140550 <https://reviews.llvm.org/D144162#4140550>, @sameerds wrote:

> I agree with @gandhi21299 ... it's best to work through the passes one at a time. But maybe we should not enable each change as it happens. We could put the changes behind a command-line option to switch between DA and UA. One single option that switches whatever passes have been updated, until all passes are updated and we are ready to flip the switch permanently.

Do you mean a single option that determines the analysis to use and all transformations use analysis depending on the option value? Something like `--enable-uniformity-analysis=1`?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144162