Skip to content

Commit 8a521cf

Browse files
committed
Removed WorkloadActionSingleInvokes from BenchmarkType and EngineParameters, just use () => engineParameters.WorkloadActionNoUnroll(1) instead.
1 parent e1c90d5 commit 8a521cf

File tree

7 files changed

+1
-41
lines changed

7 files changed

+1
-41
lines changed

src/BenchmarkDotNet/Engines/EngineFactory.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ private static Engine CreateEngine(EngineParameters engineParameters, Job job, A
112112
engineParameters.Dummy3Action,
113113
idle,
114114
main,
115-
engineParameters.WorkloadActionSingleInvoke,
115+
() => engineParameters.WorkloadActionNoUnroll(1),
116116
job,
117117
engineParameters.GlobalSetupAction,
118118
engineParameters.GlobalCleanupAction,

src/BenchmarkDotNet/Engines/EngineParameters.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ public class EngineParameters
1414
public IHost Host { get; set; }
1515
public Action<long> WorkloadActionNoUnroll { get; set; }
1616
public Action<long> WorkloadActionUnroll { get; set; }
17-
public Action WorkloadActionSingleInvoke { get; set; }
1817
public Action Dummy1Action { get; set; }
1918
public Action Dummy2Action { get; set; }
2019
public Action Dummy3Action { get; set; }

src/BenchmarkDotNet/Templates/BenchmarkType.txt

Lines changed: 0 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@
2626
Host = host,
2727
WorkloadActionUnroll = instance.WorkloadActionUnroll,
2828
WorkloadActionNoUnroll = instance.WorkloadActionNoUnroll,
29-
WorkloadActionSingleInvoke = instance.WorkloadActionSingleInvoke,
3029
Dummy1Action = instance.Dummy1,
3130
Dummy2Action = instance.Dummy2,
3231
Dummy3Action = instance.Dummy3,
@@ -150,12 +149,6 @@
150149
consumer.Consume(workloadDelegate($PassArguments$)$ConsumeField$);
151150
}
152151
}
153-
154-
private void WorkloadActionSingleInvoke()
155-
{
156-
$LoadArguments$
157-
consumer.Consume(workloadDelegate($PassArguments$)$ConsumeField$);
158-
}
159152

160153
[System.Runtime.CompilerServices.MethodImpl(System.Runtime.CompilerServices.MethodImplOptions.NoOptimization | System.Runtime.CompilerServices.MethodImplOptions.NoInlining)]
161154
public $WorkloadMethodReturnType$ $DisassemblerEntryMethodName$()
@@ -215,12 +208,6 @@
215208
NonGenericKeepAliveWithoutBoxing(result);
216209
}
217210

218-
private void WorkloadActionSingleInvoke()
219-
{
220-
$LoadArguments$
221-
NonGenericKeepAliveWithoutBoxing(workloadDelegate($PassArguments$));
222-
}
223-
224211
// we must not simply use DeadCodeEliminationHelper.KeepAliveWithoutBoxing<T> because it's generic method
225212
// and stack-only types like Span<T> can not be generic type arguments http://adamsitnik.com/Span/#span-must-not-be-a-generic-type-argument
226213
[System.Runtime.CompilerServices.MethodImpl(System.Runtime.CompilerServices.MethodImplOptions.NoInlining)]
@@ -286,14 +273,6 @@
286273
BenchmarkDotNet.Engines.DeadCodeEliminationHelper.KeepAliveWithoutBoxing(ref alias);
287274
}
288275

289-
private void WorkloadActionSingleInvoke()
290-
{
291-
$LoadArguments$
292-
ref $WorkloadMethodReturnType$ alias = ref workloadDefaultValueHolder;
293-
alias = workloadDelegate($PassArguments$);
294-
BenchmarkDotNet.Engines.DeadCodeEliminationHelper.KeepAliveWithoutBoxing(ref alias);
295-
}
296-
297276
[System.Runtime.CompilerServices.MethodImpl(System.Runtime.CompilerServices.MethodImplOptions.NoOptimization | System.Runtime.CompilerServices.MethodImplOptions.NoInlining)]
298277
public ref $WorkloadMethodReturnType$ $DisassemblerEntryMethodName$()
299278
{
@@ -353,14 +332,6 @@
353332
BenchmarkDotNet.Engines.DeadCodeEliminationHelper.KeepAliveWithoutBoxingReadonly(alias);
354333
}
355334

356-
private void WorkloadActionSingleInvoke()
357-
{
358-
$LoadArguments$
359-
ref $WorkloadMethodReturnType$ alias = ref workloadDefaultValueHolder;
360-
alias = workloadDelegate($PassArguments$);
361-
BenchmarkDotNet.Engines.DeadCodeEliminationHelper.KeepAliveWithoutBoxingReadonly(alias);
362-
}
363-
364335
[System.Runtime.CompilerServices.MethodImpl(System.Runtime.CompilerServices.MethodImplOptions.NoOptimization | System.Runtime.CompilerServices.MethodImplOptions.NoInlining)]
365336
public ref readonly $WorkloadMethodReturnType$ $DisassemblerEntryMethodName$()
366337
{
@@ -410,12 +381,6 @@
410381
}
411382
}
412383

413-
private void WorkloadActionSingleInvoke()
414-
{
415-
$LoadArguments$
416-
workloadDelegate($PassArguments$);
417-
}
418-
419384
[System.Runtime.CompilerServices.MethodImpl(System.Runtime.CompilerServices.MethodImplOptions.NoOptimization | System.Runtime.CompilerServices.MethodImplOptions.NoInlining)]
420385
public void $DisassemblerEntryMethodName$()
421386
{

src/BenchmarkDotNet/Toolchains/InProcess.Emit.Implementation/Runnable/RunnableConstants.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@ public class RunnableConstants
3535
public const string OverheadActionNoUnrollMethodName = "OverheadActionNoUnroll";
3636
public const string WorkloadActionUnrollMethodName = "WorkloadActionUnroll";
3737
public const string WorkloadActionNoUnrollMethodName = "WorkloadActionNoUnroll";
38-
public const string WorkloadActionSingleInvokeMethodName = "WorkloadActionSingleInvoke";
3938
public const string ForDisassemblyDiagnoserMethodName = "__ForDisassemblyDiagnoser__";
4039
public const string InvokeCountParamName = "invokeCount";
4140

src/BenchmarkDotNet/Toolchains/InProcess.Emit.Implementation/Runnable/RunnableReuse.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,6 @@ private static EngineParameters CreateEngineParameters<T>(
9090
Host = host,
9191
WorkloadActionUnroll = LoopCallbackFromMethod(instance, WorkloadActionUnrollMethodName),
9292
WorkloadActionNoUnroll = LoopCallbackFromMethod(instance, WorkloadActionNoUnrollMethodName),
93-
WorkloadActionSingleInvoke = CallbackFromMethod(instance, WorkloadActionSingleInvokeMethodName),
9493
Dummy1Action = CallbackFromMethod(instance, Dummy1MethodName),
9594
Dummy2Action = CallbackFromMethod(instance, Dummy2MethodName),
9695
Dummy3Action = CallbackFromMethod(instance, Dummy3MethodName),

src/BenchmarkDotNet/Toolchains/InProcess.NoEmit/InProcessNoEmitRunner.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,6 @@ public static void RunCore(IHost host, BenchmarkCase benchmarkCase)
135135
workloadAction.InvokeSingle();
136136
},
137137
WorkloadActionUnroll = workloadAction.InvokeMultiple,
138-
WorkloadActionSingleInvoke = workloadAction.InvokeSingle,
139138
Dummy1Action = dummy1.InvokeSingle,
140139
Dummy2Action = dummy2.InvokeSingle,
141140
Dummy3Action = dummy3.InvokeSingle,

src/BenchmarkDotNet/Toolchains/InProcess/InProcessRunner.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,6 @@ public static void RunCore(IHost host, BenchmarkCase benchmarkCase, BenchmarkAct
134134
workloadAction.InvokeSingle();
135135
},
136136
WorkloadActionUnroll = workloadAction.InvokeMultiple,
137-
WorkloadActionSingleInvoke = workloadAction.InvokeSingle,
138137
Dummy1Action = dummy1.InvokeSingle,
139138
Dummy2Action = dummy2.InvokeSingle,
140139
Dummy3Action = dummy3.InvokeSingle,

0 commit comments

Comments
 (0)