[all-commits] [llvm/llvm-project] 6f288b: [llvm-reduce] Count chunks by running a preliminar...

[aeubanks via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6f288bd7727e6906f6e85208d710b70acf598514
      https://github.com/llvm/llvm-project/commit/6f288bd7727e6906f6e85208d710b70acf598514
  Author: Arthur Eubanks <aeubanks at google.com>
  Date:   2021-11-11 (Thu, 11 Nov 2021)

  Changed paths:
    M llvm/tools/llvm-reduce/deltas/Delta.cpp
    M llvm/tools/llvm-reduce/deltas/Delta.h
    M llvm/tools/llvm-reduce/deltas/ReduceAliases.cpp
    M llvm/tools/llvm-reduce/deltas/ReduceArguments.cpp
    M llvm/tools/llvm-reduce/deltas/ReduceAttributes.cpp
    M llvm/tools/llvm-reduce/deltas/ReduceBasicBlocks.cpp
    M llvm/tools/llvm-reduce/deltas/ReduceFunctionBodies.cpp
    M llvm/tools/llvm-reduce/deltas/ReduceFunctions.cpp
    M llvm/tools/llvm-reduce/deltas/ReduceGlobalObjects.cpp
    M llvm/tools/llvm-reduce/deltas/ReduceGlobalValues.cpp
    M llvm/tools/llvm-reduce/deltas/ReduceGlobalVarInitializers.cpp
    M llvm/tools/llvm-reduce/deltas/ReduceGlobalVars.cpp
    M llvm/tools/llvm-reduce/deltas/ReduceInstructions.cpp
    M llvm/tools/llvm-reduce/deltas/ReduceInstructionsMIR.cpp
    M llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp
    M llvm/tools/llvm-reduce/deltas/ReduceModuleData.cpp
    M llvm/tools/llvm-reduce/deltas/ReduceOperandBundles.cpp
    M llvm/tools/llvm-reduce/deltas/ReduceOperands.cpp
    M llvm/tools/llvm-reduce/deltas/ReduceOperandsToArgs.cpp
    M llvm/tools/llvm-reduce/deltas/ReduceSpecialGlobals.cpp

  Log Message:
  -----------
  [llvm-reduce] Count chunks by running a preliminary reduction

Having a separate counting method runs the risk of a mismatch between
the actual reduction method and the counting method.

Instead, create an Oracle that always returns true for shouldKeep(), run
the reduction, and count how many times shouldKeep() was called. The
module should not be modified if shouldKeep() always returns true.

Reviewed By: Meinersbur

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