[PATCH] D111257: [fir] Add affine demotion pass

Valentin Clement via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 7 09:06:23 PDT 2021


clementval added a comment.

In D111257#3047108 <https://reviews.llvm.org/D111257#3047108>, @mehdi_amini wrote:

> LG.
>
> Something I just noticed is that the passes in general are lacking a consistent prefix: there is a risk of collision when used later in a library. It'd be great to standardize this (outside of this CL).

Do you mean prefixing the class name? I have tried to look at the core MLIR and I don't see a pattern for dialects passes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111257



More information about the llvm-commits mailing list