[all-commits] [llvm/llvm-project] b1a958: [opt] Disincentivize new tests from using old pass...

Roman Lebedev via All-commits all-commits at lists.llvm.org
Thu Dec 8 12:54:36 PST 2022 

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b1a95848180e66758f7a6d5bd172ced11a2e8914
      https://github.com/llvm/llvm-project/commit/b1a95848180e66758f7a6d5bd172ced11a2e8914
  Author: Roman Lebedev <lebedev.ri at gmail.com>
  Date:   2022-12-08 (Thu, 08 Dec 2022)

  Changed paths:
    M clang/test/CodeGen/arm-neon-vcadd.c
    M clang/test/CodeGen/memcmp-inline-builtin-to-asm.c
    M clang/test/CodeGenCXX/attr-musttail.cpp
    M llvm/test/Analysis/AliasSet/unknown-inst-tracking.ll
    M llvm/test/Analysis/CallGraph/2008-09-09-DirectCall.ll
    M llvm/test/Analysis/CallGraph/2008-09-09-UsedByGlobal.ll
    M llvm/test/Analysis/CallGraph/callback-calls.ll
    M llvm/test/Analysis/CallGraph/ignore-assumelike-calls.ll
    M llvm/test/Analysis/CallGraph/ignore-callback-uses.ll
    M llvm/test/Analysis/CallGraph/llvm-used.ll
    M llvm/test/Analysis/CallGraph/no-intrinsics.ll
    M llvm/test/Analysis/CallGraph/non-leaf-intrinsics.ll
    M llvm/test/Analysis/Dominators/2007-07-11-SplitBlock.ll
    M llvm/test/Analysis/Dominators/2007-07-12-SplitBlock.ll
    M llvm/test/Analysis/Dominators/invoke.ll
    M llvm/test/Analysis/GlobalsModRef/pr12351.ll
    M llvm/test/Analysis/LazyValueAnalysis/lvi-after-jumpthreading.ll
    M llvm/test/Analysis/LazyValueAnalysis/lvi-for-ashr.ll
    M llvm/test/Analysis/LoopAccessAnalysis/interleave-innermost.ll
    M llvm/test/Analysis/LoopAccessAnalysis/memcheck-ni.ll
    M llvm/test/Analysis/LoopAccessAnalysis/scalable-vector-regression-tests.ll
    M llvm/test/Analysis/MemorySSA/debugvalue.ll
    M llvm/test/Analysis/MemorySSA/loop-rotate-inv-template.ll
    M llvm/test/Analysis/MemorySSA/loop-rotate-simplified-clone.ll
    M llvm/test/Analysis/MemorySSA/loop-rotate-valuemap.ll
    M llvm/test/Analysis/MemorySSA/loop-unswitch.ll
    M llvm/test/Analysis/MemorySSA/nondeterminism.ll
    M llvm/test/Analysis/MemorySSA/pr40037.ll
    M llvm/test/Analysis/MemorySSA/pr40749_2.ll
    M llvm/test/Analysis/MemorySSA/pr42294.ll
    M llvm/test/Analysis/MemorySSA/pr43044.ll
    M llvm/test/Analysis/MemorySSA/pr43641.ll
    M llvm/test/Analysis/MemorySSA/pr44029.ll
    M llvm/test/Analysis/MemorySSA/pr46574.ll
    M llvm/test/Analysis/MemorySSA/simple_loop_unswitch_nontrivial.ll
    M llvm/test/Analysis/MustExecute/const-cond.ll
    M llvm/test/Analysis/MustExecute/infinite_loops.ll
    M llvm/test/Analysis/MustExecute/loop-header.ll
    M llvm/test/Analysis/MustExecute/pr57780.ll
    M llvm/test/Analysis/ScalarEvolution/2007-08-06-MisinterpretBranch.ll
    M llvm/test/Analysis/ScalarEvolution/2012-03-26-LoadConstant.ll
    M llvm/test/Analysis/ScalarEvolution/2012-05-29-MulAddRec.ll
    M llvm/test/Analysis/ScalarEvolution/avoid-assume-hang.ll
    M llvm/test/Analysis/ScalarEvolution/scev-expander-existing-value-offset.ll
    M llvm/test/Analysis/ScalarEvolution/scev-expander-reuse-unroll.ll
    M llvm/test/Analysis/ScalarEvolution/scev-expander-reuse-vect.ll
    M llvm/test/Analysis/ScalarEvolution/scev-invalid.ll
    M llvm/test/Bitcode/compatibility-6.0.ll
    M llvm/test/Bitcode/extractelement.ll
    M llvm/test/Bitcode/infer_dso_local.ll
    M llvm/test/CodeGen/AArch64/llvm-masked-gather-legal-for-sve.ll
    M llvm/test/CodeGen/AArch64/llvm-masked-scatter-legal-for-sve.ll
    M llvm/test/CodeGen/AArch64/sve-ptest-removal-cmple.ll
    M llvm/test/CodeGen/AMDGPU/amdgpu-unroll-threshold.ll
    M llvm/test/CodeGen/AMDGPU/force-alwaysinline-lds-global-address.ll
    M llvm/test/CodeGen/AMDGPU/inline-maxbb.ll
    M llvm/test/CodeGen/AMDGPU/loop-idiom.ll
    M llvm/test/CodeGen/AMDGPU/opencl-printf-and-hostcall.ll
    M llvm/test/CodeGen/AMDGPU/opencl-printf.ll
    M llvm/test/CodeGen/AMDGPU/propagate-attributes-function-pointer-argument.ll
    M llvm/test/CodeGen/AMDGPU/r600.alu-limits.ll
    M llvm/test/CodeGen/AMDGPU/reqd-work-group-size.ll
    M llvm/test/CodeGen/AMDGPU/simplify-libcalls2.ll
    M llvm/test/CodeGen/AMDGPU/speculative-execution-freecasts.ll
    M llvm/test/CodeGen/AMDGPU/stress-calls.ll
    M llvm/test/CodeGen/AMDGPU/tti-unroll-prefs.ll
    M llvm/test/CodeGen/AMDGPU/unify-metadata.ll
    M llvm/test/CodeGen/AMDGPU/vectorize-buffer-fat-pointer.ll
    M llvm/test/CodeGen/ARM/bit-reverse-to-rbit.ll
    M llvm/test/CodeGen/Generic/overloaded-intrinsic-name.ll
    M llvm/test/CodeGen/NVPTX/access-non-generic.ll
    M llvm/test/CodeGen/NVPTX/intrinsic-old.ll
    M llvm/test/CodeGen/NVPTX/math-intrins-sm80-ptx70-instcombine.ll
    M llvm/test/CodeGen/NVPTX/nvvm-reflect-module-flag.ll
    M llvm/test/CodeGen/NVPTX/speculative-execution-divergent-target.ll
    M llvm/test/CodeGen/PowerPC/ppc64-P9-mod.ll
    M llvm/test/CodeGen/PowerPC/vec_shuffle.ll
    M llvm/test/CodeGen/PowerPC/vperm-instcombine.ll
    M llvm/test/CodeGen/SystemZ/inline-thresh-adjust.ll
    M llvm/test/CodeGen/Thumb/consthoist-few-dependents.ll
    M llvm/test/CodeGen/Thumb2/mve-intrinsics/predicates.ll
    M llvm/test/CodeGen/Thumb2/mve-intrinsics/vadc-multiple.ll
    M llvm/test/CodeGen/Thumb2/mve-vpt-from-intrinsics.ll
    M llvm/test/CodeGen/WinEH/wineh-intrinsics.ll
    M llvm/test/CodeGen/X86/GC/deadargelim.ll
    M llvm/test/CodeGen/X86/GC/inline.ll
    M llvm/test/CodeGen/X86/GC/inline2.ll
    M llvm/test/CodeGen/X86/block-placement.ll
    M llvm/test/CodeGen/X86/constant-hoisting-bfi.ll
    M llvm/test/CodeGen/X86/deopt-intrinsic-cconv.ll
    M llvm/test/CodeGen/X86/frame-order.ll
    M llvm/test/CodeGen/X86/loadStore_vectorizer.ll
    M llvm/test/CodeGen/X86/masked_gather_scatter.ll
    M llvm/test/CodeGen/X86/no-plt-libcalls.ll
    M llvm/test/CodeGen/X86/pointer-vector.ll
    M llvm/test/CodeGen/X86/pr38762.ll
    M llvm/test/CodeGen/X86/pr38763.ll
    M llvm/test/CodeGen/X86/pr39243.ll
    M llvm/test/CodeGen/X86/tls-loads-control2.ll
    M llvm/test/CodeGen/X86/vector-gep.ll
    M llvm/test/DebugInfo/AArch64/ir-outliner.ll
    M llvm/test/DebugInfo/ARM/lowerbdgdeclare_vla.ll
    M llvm/test/DebugInfo/Generic/2010-03-19-DbgDeclare.ll
    M llvm/test/DebugInfo/Generic/PR37395.ll
    M llvm/test/DebugInfo/Generic/assignment-tracking/dse/dse-after-memcpyopt-merge.ll
    M llvm/test/DebugInfo/Generic/assignment-tracking/dse/shorten.ll
    M llvm/test/DebugInfo/Generic/assignment-tracking/licm/merge.ll
    M llvm/test/DebugInfo/Generic/assignment-tracking/licm/multi-exit.ll
    M llvm/test/DebugInfo/Generic/assignment-tracking/memcpyopt/merge-stores.ll
    M llvm/test/DebugInfo/Generic/assignment-tracking/parse-and-verify/instruction-type.ll
    M llvm/test/DebugInfo/Generic/assignment-tracking/parse-and-verify/roundtrip.ll
    M llvm/test/DebugInfo/Generic/assignment-tracking/parse-and-verify/verify.ll
    M llvm/test/DebugInfo/Generic/callsite-attr-invalid.ll
    M llvm/test/DebugInfo/Generic/debug-info-always-inline.ll
    M llvm/test/DebugInfo/Generic/debug_value_list.ll
    M llvm/test/DebugInfo/Generic/global-sra-array.ll
    M llvm/test/DebugInfo/Generic/global-sra-single-member.ll
    M llvm/test/DebugInfo/Generic/global-sra-struct-fwd-decl.ll
    M llvm/test/DebugInfo/Generic/global-sra-struct-zero-length.ll
    M llvm/test/DebugInfo/Generic/global-sra-struct.ll
    M llvm/test/DebugInfo/Generic/indvar-discriminator.ll
    M llvm/test/DebugInfo/Generic/inline-debug-info-multiret.ll
    M llvm/test/DebugInfo/Generic/inline-debug-info.ll
    M llvm/test/DebugInfo/Generic/inline-debug-loc.ll
    M llvm/test/DebugInfo/Generic/inline-no-debug-info.ll
    M llvm/test/DebugInfo/Generic/licm-hoist-debug-loc.ll
    M llvm/test/DebugInfo/Generic/licm-hoist-intrinsic-debug-loc.ll
    M llvm/test/DebugInfo/Generic/simplifycfg_sink_last_inst.ll
    M llvm/test/DebugInfo/Generic/volatile-alloca.ll
    M llvm/test/DebugInfo/X86/clone-module-2.ll
    M llvm/test/DebugInfo/X86/clone-module.ll
    M llvm/test/DebugInfo/X86/dbg-value-dropped-instcombine.ll
    M llvm/test/DebugInfo/X86/dbgloc-insert-extract-val-instrs.ll
    M llvm/test/DebugInfo/X86/dead-store-elimination-marks-undef.ll
    M llvm/test/DebugInfo/X86/global-sra-fp80-array.ll
    M llvm/test/DebugInfo/X86/global-sra-fp80-struct.ll
    M llvm/test/DebugInfo/X86/global-sra-struct-fit-segment.ll
    M llvm/test/DebugInfo/X86/instcombine-demanded-bits-salvage.ll
    M llvm/test/DebugInfo/X86/licm-undef-dbg-value.ll
    M llvm/test/DebugInfo/X86/undef-type-md.ll
    M llvm/test/DebugInfo/check-debugify-preserves-analyses.ll
    M llvm/test/DebugInfo/debugify-bogus-dbg-value.ll
    M llvm/test/DebugInfo/debugify-export.ll
    M llvm/test/DebugInfo/debugify-ignore-phi.ll
    M llvm/test/DebugInfo/debugify-original-no-dbg-info.ll
    M llvm/test/DebugInfo/debugify-report-missing-locs-only.ll
    M llvm/test/DebugInfo/debugify.ll
    M llvm/test/DebugInfo/duplicate_dbgvalue.ll
    M llvm/test/DebugInfo/limit-arglist-size.ll
    M llvm/test/DebugInfo/pr34672.ll
    M llvm/test/DebugInfo/return-same-line-merge.ll
    M llvm/test/DebugInfo/salvage-duplicate-values.ll
    M llvm/test/DebugInfo/salvage-gep.ll
    M llvm/test/DebugInfo/salvage-limit-expr-size.ll
    M llvm/test/DebugInfo/salvage-nonconst-binop.ll
    M llvm/test/DebugInfo/simplify-cfg-preserve-dbg-values.ll
    M llvm/test/DebugInfo/unrolled-loop-remainder.ll
    M llvm/test/DebugInfo/verify-di-preserve.ll
    M llvm/test/Feature/OperandBundles/inliner-conservative.ll
    M llvm/test/Feature/OperandBundles/inliner-funclet-wineh.ll
    M llvm/test/Feature/OperandBundles/merge-func.ll
    M llvm/test/Feature/global_pv.ll
    M llvm/test/LTO/X86/keep-used-puts-during-instcombine.ll
    M llvm/test/Other/2002-02-24-InlineBrokePHINodes.ll
    M llvm/test/Other/2007-09-10-PassManager.ll
    M llvm/test/Other/2008-02-14-PassManager.ll
    M llvm/test/Other/2008-06-04-FieldSizeInPacked.ll
    M llvm/test/Other/2008-10-06-RemoveDeadPass.ll
    M llvm/test/Other/2009-06-05-no-implicit-float.ll
    M llvm/test/Other/X86/debugcounter-divrempairs.ll
    M llvm/test/Other/X86/debugcounter-partiallyinlinelibcalls.ll
    M llvm/test/Other/cfg-printer-branch-weights-percent.ll
    M llvm/test/Other/cfg-printer-branch-weights.ll
    M llvm/test/Other/cfg-printer-filter.ll
    M llvm/test/Other/cfg_deopt_unreach.ll
    M llvm/test/Other/constant-fold-gep.ll
    M llvm/test/Other/copy-metadata-of-declaration.ll
    M llvm/test/Other/debugcounter-dce.ll
    M llvm/test/Other/debugcounter-newgvn.ll
    M llvm/test/Other/heat-colors-graphs.ll
    M llvm/test/Other/heat-colors-multigraph.ll
    M llvm/test/Other/module-pass-printer.ll
    M llvm/test/Other/opt-legacy-syntax-deprecation.ll
    M llvm/test/Other/optimization-remarks-inline.ll
    M llvm/test/Other/pr32085.ll
    M llvm/test/Other/print-debug-counter.ll
    M llvm/test/Other/scalable-vector-struct-intrinsic.ll
    M llvm/test/Other/scalable-vectors-core-ir.ll
    M llvm/test/Transforms/HardwareLoops/ARM/structure.ll
    M llvm/test/Transforms/LoadStoreVectorizer/X86/codegenprepare-produced-address-math.ll
    M llvm/test/Verifier/dbg-invalid-vector.ll
    M llvm/test/Verifier/di-subroutine-localvar.ll
    M llvm/test/Verifier/fp-intrinsics.ll
    M llvm/test/Verifier/non-integral-pointers.ll
    M llvm/test/Verifier/preallocated-valid.ll
    M llvm/test/Verifier/recursive-struct-param.ll
    M llvm/test/Verifier/statepoint.ll
    M llvm/test/Verifier/swifttailcc-musttail-valid.ll
    M llvm/test/Verifier/tbaa.ll
    M llvm/test/Verifier/vp-intrinsics.ll
    M llvm/test/Verifier/weak-dllimport.ll
    M llvm/test/tools/UpdateTestChecks/update_analyze_test_checks/Inputs/x86-loopvectorize-costmodel.ll
    M llvm/test/tools/UpdateTestChecks/update_analyze_test_checks/Inputs/x86-loopvectorize-costmodel.ll.expected
    M llvm/test/tools/llvm-reduce/remove-function-bodies-comdat.ll
    M llvm/tools/opt/opt.cpp

  Log Message:
  -----------
  [opt] Disincentivize new tests from using old pass syntax

Over the past day or so, i've took a large swing at our tests,
and reduced the number of tests that were still using the old syntax
from ~1800 to just 200.

Left to handle: (as it is seen in this patch)
* Transforms/LSR
* Transforms/CGP
* Transforms/TypePromotion
* Transforms/HardwareLoops
* Analysis/*
* some misc.

I think this is the right point to start actively refusing
to honor the old syntax, except for the old tests,
to prevent the old syntax from creeping back in.

Thus, let's add temporary default-off flag,
and if it is not passed refuse to accept old syntax.
The tests that still need porting are annotated with this flag.

Reviewed By: aeubanks

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

More information about the All-commits mailing list