Added pointers to SampleBenchmark test.

Added comment about `Type.IsClass` returns true for pointers.
dotnet · May 17, 2023 · 48c10d0 · 48c10d0
1 parent 7abb119
commit 48c10d0
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 0 deletions.
diff --git a/src/BenchmarkDotNet/Engines/Consumer.cs b/src/BenchmarkDotNet/Engines/Consumer.cs
@@ -131,6 +131,7 @@ public void Consume<T>(in T value)
             => byteHolder = Unsafe.As<T, byte>(ref Unsafe.AsRef(in value));
 
         internal static bool IsConsumable(Type type)
+            // IsClass returns true for pointers.
             => SupportedTypes.Contains(type) || type.GetTypeInfo().IsClass || type.GetTypeInfo().IsInterface || !type.IsByRefLike();
 
         internal static bool HasConsumableField(Type type, out FieldInfo consumableField)

diff --git a/tests/BenchmarkDotNet.IntegrationTests/InProcess.EmitTests/SampleBenchmark.cs b/tests/BenchmarkDotNet.IntegrationTests/InProcess.EmitTests/SampleBenchmark.cs
@@ -43,6 +43,27 @@ public EmptyStruct ReturnManyArgsCase(ref double i, int j, string k, object l)
             return ref refValueHolder;
         }
 
+        [Benchmark]
+        public unsafe int* ReturnsIntPointer()
+        {
+            Thread.Sleep(100);
+            return default;
+        }
+
+        [Benchmark]
+        public unsafe void* ReturnsVoidPointer()
+        {
+            Thread.Sleep(100);
+            return default;
+        }
+
+        [Benchmark]
+        public unsafe EmptyStruct* ReturnsStructPointer()
+        {
+            Thread.Sleep(100);
+            return default;
+        }
+
         [Benchmark, Arguments(12)]
         public Task<int> TaskSample(long arg)
         {