[all-commits] [llvm/llvm-project] 7e25b2: IRNormalizer: Replace cl::opts with pass parameter...

[Matt Arsenault via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7e25b240731413d2cfca2b78ab1d0ed33d851622
      https://github.com/llvm/llvm-project/commit/7e25b240731413d2cfca2b78ab1d0ed33d851622
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2025-04-01 (Tue, 01 Apr 2025)

  Changed paths:
    M llvm/include/llvm/Transforms/Utils/IRNormalizer.h
    M llvm/lib/Passes/PassBuilder.cpp
    M llvm/lib/Passes/PassRegistry.def
    M llvm/lib/Transforms/Utils/IRNormalizer.cpp
    A llvm/test/Transforms/IRNormalizer/pass-parameters.ll
    M llvm/test/Transforms/IRNormalizer/reordering.ll

  Log Message:
  -----------
  IRNormalizer: Replace cl::opts with pass parameters (#133874)

Not sure why the "fold-all" option naming didn't match the
variable "FoldPreOutputs", but I've preserved the difference.

More annoyingly, the pass name "normalize" does not match the pass
name IRNormalizer and should probably be fixed one way or the other.

Also the existing test coverage for the flags is lacking. I've added
a test that shows they parse, but we should have tests that they
do something.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications