[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