[all-commits] [llvm/llvm-project] ebad5f: [mlir][Canonicalize] Fix command-line options

rk via All-commits all-commits at lists.llvm.org
Wed May 18 00:38:13 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: ebad5fb309570765e8f121c441dcd90b5aa0536a
      https://github.com/llvm/llvm-project/commit/ebad5fb309570765e8f121c441dcd90b5aa0536a
  Author: rkayaith <rkayaith at gmail.com>
  Date:   2022-05-18 (Wed, 18 May 2022)

  Changed paths:
    M mlir/lib/Transforms/Canonicalizer.cpp
    M mlir/test/Transforms/test-canonicalize.mlir

  Log Message:
  -----------
  [mlir][Canonicalize] Fix command-line options

The canonicalize command-line options currently have no effect, as the pass is
reading the pass options in its constructor, before they're actually
initialized. This results in the default values of the options always being used.

The change here moves the initialization of the `GreedyRewriteConfig` out of the
constructor, so that it runs after the pass options have been parsed.

Fixes #55466

Reviewed By: rriddle

Differential Revision: https://reviews.llvm.org/D125621




More information about the All-commits mailing list