[PATCH] D105627: [NewPM] Consistently use 'simplifycfg' rather than 'simplify-cfg'

Bjorn Pettersson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 8 06:23:55 PDT 2021


bjope created this revision.
bjope added reviewers: ychen, aeubanks.
Herald added subscribers: ormris, wenlei, lxfind, asbirlea, george.burgess.iv, zzheng, haicheng, hiraditya, eraman.
bjope requested review of this revision.
Herald added a project: LLVM.

There was an alias between 'simplifycfg' and 'simplify-cfg' in the
PassRegistry. That was the original reason for this patch, which
effectively removes the alias.

This patch also replaces all occurrances of 'simplify-cfg'
by 'simplifycfg'. Reason for choosing that form for the name is
that it matches the DEBUG_TYPE for the pass, and the legacy PM name
and also how it is spelled out in other passes such as
'loop-simplifycfg', and in other options such as
'simplifycfg-merge-cond-stores'.

I for some reason the name should be changed to 'simplify-cfg' in
the future, then I think such a renaming should be more widely done
and not only impacting the PassRegistry.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D105627

Files:
  llvm/lib/Analysis/InlineCost.cpp
  llvm/lib/FuzzMutate/FuzzerCLI.cpp
  llvm/lib/Passes/PassBuilder.cpp
  llvm/lib/Passes/PassRegistry.def
  llvm/lib/Transforms/IPO/PassManagerBuilder.cpp
  llvm/lib/Transforms/Scalar/SimpleLoopUnswitch.cpp
  llvm/test/Analysis/MemorySSA/update-remove-dead-blocks.ll
  llvm/test/Other/cgscc-disconnected-invalidation.ll
  llvm/test/Other/cgscc-iterate-function-mutation.ll
  llvm/test/Other/pass-pipelines.ll
  llvm/test/Other/print-module-scope.ll
  llvm/test/Transforms/Coroutines/coro-heap-elide.ll
  llvm/test/Transforms/Inline/monster_scc.ll
  llvm/test/Transforms/InstCombine/assume-align.ll
  llvm/test/Transforms/LoopUnroll/peel-loop-inner.ll
  llvm/test/Transforms/LoopUnroll/peel-loop.ll
  llvm/test/Transforms/PGOProfile/chr.ll
  llvm/test/Transforms/PruneEH/2008-06-02-Weak.ll
  llvm/test/Transforms/PruneEH/ipo-nounwind.ll
  llvm/test/Transforms/PruneEH/looptest.ll
  llvm/test/Transforms/PruneEH/musttail.ll
  llvm/test/Transforms/PruneEH/operand-bundles.ll
  llvm/test/Transforms/PruneEH/pr23971.ll
  llvm/test/Transforms/PruneEH/pr26263.ll
  llvm/test/Transforms/PruneEH/recursivetest.ll
  llvm/test/Transforms/PruneEH/seh-nounwind.ll
  llvm/test/Transforms/PruneEH/simpletest.ll
  llvm/test/Transforms/SimplifyCFG/1elt-ptr-vec-alignment-crash.ll
  llvm/test/Transforms/SimplifyCFG/ARM/switch-to-lookup-table.ll
  llvm/test/Transforms/SimplifyCFG/ForwardSwitchConditionToPHI.ll
  llvm/test/Transforms/SimplifyCFG/X86/CoveredLookupTable.ll
  llvm/test/Transforms/SimplifyCFG/X86/disable-lookup-table.ll
  llvm/test/Transforms/SimplifyCFG/X86/sink-common-code.ll
  llvm/test/Transforms/SimplifyCFG/X86/switch-covered-bug.ll
  llvm/test/Transforms/SimplifyCFG/X86/switch-table-bug.ll
  llvm/test/Transforms/SimplifyCFG/X86/switch_to_lookup_table.ll
  llvm/test/Transforms/SimplifyCFG/basictest.ll
  llvm/test/Transforms/SimplifyCFG/branch-fold-threshold.ll
  llvm/test/Transforms/SimplifyCFG/convergent.ll
  llvm/test/Transforms/SimplifyCFG/multiple-phis.ll
  llvm/test/Transforms/SimplifyCFG/no-md-sink.ll
  llvm/test/Transforms/SimplifyCFG/preserve-llvm-loop-metadata.ll
  llvm/test/Transforms/SimplifyCFG/rangereduce.ll
  llvm/test/Transforms/SimplifyCFG/switch-dead-default.ll
  llvm/test/Transforms/SimplifyCFG/switch_undef.ll
  llvm/test/Transforms/SimplifyCFG/unprofitable-pr.ll
  llvm/test/Transforms/SimplifyCFG/wc-widen-block.ll
  llvm/tools/bugpoint/CrashDebugger.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D105627.357205.patch
Type: text/x-patch
Size: 37759 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210708/eb401b04/attachment.bin>


More information about the llvm-commits mailing list