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
Stop using value tuples.
  • Loading branch information
@teo-tsirpanis
teo-tsirpanis committed Oct 27, 2022
commit 5001e7fa9e3da92f8840ef472fd8ef6170baea58
23 changes: 12 additions & 11 deletions src/libraries/System.Reflection.Metadata/src/System/Reflection/Metadata/BlobContentId.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,13 +12,16 @@ namespace System.Reflection.Metadata
{
private const int Size = BlobUtilities.SizeOfGuid + sizeof(uint);

public Guid Guid { get; }
public uint Stamp { get; }
private readonly Guid _guid;
private readonly uint _stamp;

public Guid Guid => _guid;
public uint Stamp => _stamp;

public BlobContentId(Guid guid, uint stamp)
{
Guid = guid;
Stamp = stamp;
_guid = guid;
_stamp = stamp;
}

public BlobContentId(ImmutableArray<byte> id)
Expand All @@ -28,7 +31,7 @@ public BlobContentId(ImmutableArray<byte> id)
Throw.ArgumentNull(nameof(id));
}

(Guid, Stamp) = Initialize(id.AsSpan());
Initialize(id.AsSpan(), out _guid, out _stamp);
}

public BlobContentId(byte[] id)
Expand All @@ -38,10 +41,10 @@ public BlobContentId(byte[] id)
Throw.ArgumentNull(nameof(id));
}

(Guid, Stamp) = Initialize(id);
Initialize(id, out _guid, out _stamp);
}

private static unsafe (Guid guid, uint stamp) Initialize(ReadOnlySpan<byte> id)
private static unsafe void Initialize(ReadOnlySpan<byte> id, out Guid guid, out uint stamp)
{
if (id.Length != Size)
{
Expand All @@ -51,10 +54,8 @@ private static unsafe (Guid guid, uint stamp) Initialize(ReadOnlySpan<byte> id)
fixed (byte* ptr = &id[0])
{
var reader = new BlobReader(ptr, id.Length);
Guid guid = reader.ReadGuid();
uint stamp = reader.ReadUInt32();

return (guid, stamp);
guid = reader.ReadGuid();
stamp = reader.ReadUInt32();
}
}

Expand Down