Skip to content

Use more spans in System.Reflection.Metadata et. al. #76574

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
stephentoub merged 16 commits into from
Nov 3, 2022
Merged

Use more spans in System.Reflection.Metadata et. al. #76574

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
f6b854b
Use Unsafe.As in ImmutableByteArrayInterop.
teo-tsirpanis Oct 3, 2022
417f538
Optimize methods in the BlobUtilities class.
teo-tsirpanis Oct 3, 2022
ce584cd
Include more information in exceptions thrown from catch blocks.
teo-tsirpanis Oct 3, 2022
44e38dc
Use spans in BlobContentId, avoiding ImmutableByteArrayInterop.
teo-tsirpanis Oct 3, 2022
a2acd50
Use spans in BlobWriter, reducing pinning and ImmutableArrayInterop.
teo-tsirpanis Oct 3, 2022
daed179
Use spans in BlobBuilder, reducing pinning and ImmutableArrayInterop.
teo-tsirpanis Oct 3, 2022
47350a7
Remove NoInlining from the throw helpers.
teo-tsirpanis Oct 3, 2022
9a56ab1
Inline `BlobUtilities.Read(Immutable)?Bytes`.
teo-tsirpanis Oct 7, 2022
8ebbbfa
Avoid a length check introduced by the use of `BinaryPrimitives`.
teo-tsirpanis Oct 8, 2022
85bc2d6
Remove `DisallowNull` from `ImmutableArrayInterop.DangerousCreateFrom…
teo-tsirpanis Oct 8, 2022
de0c6f2
Remove an early bail-out if the buffer was empty.
teo-tsirpanis Oct 8, 2022
5001e7f
Stop using value tuples.
teo-tsirpanis Oct 12, 2022
94b72c1
Fix tests and expose writing a span to BlobWriter.
teo-tsirpanis Oct 12, 2022
a4b647c
Fix stack overflows.
teo-tsirpanis Oct 13, 2022
d466b38
Shorten BlobUtilities methods that write integers.
teo-tsirpanis Oct 27, 2022
bc43334
Optimize some methods in `MemoryBlock`.
teo-tsirpanis Oct 28, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Avoid a length check introduced by the use of BinaryPrimitives.
  • Loading branch information
@teo-tsirpanis
teo-tsirpanis committed Oct 27, 2022
commit 8ebbbfa6cd59e759dbd0495465e982b54311b737
39 changes: 26 additions & 13 deletions ...ries/System.Reflection.Metadata/src/System/Reflection/Internal/Utilities/BlobUtilities.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
using System.Diagnostics;
using System.Reflection.Internal;
using System.Runtime.CompilerServices;
using System.Runtime.InteropServices;

namespace System.Reflection
{
Expand All @@ -19,20 +20,12 @@ public static void WriteBytes(this byte[] buffer, int start, byte value, int byt

public static void WriteDouble(this byte[] buffer, int start, double value)
{
#if NETCOREAPP
BinaryPrimitives.WriteDoubleLittleEndian(buffer.AsSpan(start), value);
#else
WriteUInt64(buffer, start, *(ulong*)&value);
#endif
}

public static void WriteSingle(this byte[] buffer, int start, float value)
{
#if NETCOREAPP
BinaryPrimitives.WriteSingleLittleEndian(buffer.AsSpan(start), value);
#else
WriteUInt32(buffer, start, *(uint*)&value);
#endif
}

public static void WriteByte(this byte[] buffer, int start, byte value)
Expand All @@ -43,27 +36,47 @@ public static void WriteByte(this byte[] buffer, int start, byte value)

public static void WriteUInt16(this byte[] buffer, int start, ushort value)
{
BinaryPrimitives.WriteUInt16LittleEndian(buffer.AsSpan(start), value);
if (!BitConverter.IsLittleEndian)
{
value = BinaryPrimitives.ReverseEndianness(value);
}
Unsafe.WriteUnaligned(ref buffer[start], value);
}

public static void WriteUInt16BE(this byte[] buffer, int start, ushort value)
{
BinaryPrimitives.WriteUInt16BigEndian(buffer.AsSpan(start), value);
if (BitConverter.IsLittleEndian)
{
value = BinaryPrimitives.ReverseEndianness(value);
}
Unsafe.WriteUnaligned(ref buffer[start], value);
}

public static void WriteUInt32BE(this byte[] buffer, int start, uint value)
{
BinaryPrimitives.WriteUInt32BigEndian(buffer.AsSpan(start), value);
if (BitConverter.IsLittleEndian)
{
value = BinaryPrimitives.ReverseEndianness(value);
}
Unsafe.WriteUnaligned(ref buffer[start], value);
}

public static void WriteUInt32(this byte[] buffer, int start, uint value)
{
BinaryPrimitives.WriteUInt32LittleEndian(buffer.AsSpan(start), value);
if (!BitConverter.IsLittleEndian)
{
value = BinaryPrimitives.ReverseEndianness(value);
}
Unsafe.WriteUnaligned(ref buffer[start], value);
}

public static void WriteUInt64(this byte[] buffer, int start, ulong value)
{
BinaryPrimitives.WriteUInt64LittleEndian(buffer.AsSpan(start), value);
if (!BitConverter.IsLittleEndian)
{
value = BinaryPrimitives.ReverseEndianness(value);
}
Unsafe.WriteUnaligned(ref buffer[start], value);
}

public const int SizeOfSerializedDecimal = sizeof(byte) + 3 * sizeof(uint);
Expand Down