Skip to content

Commit

Permalink
addressing review comments
Browse files Browse the repository at this point in the history
  • Loading branch information
khushal1996 committed Jul 13, 2023
1 parent f56f612 commit a995aae
Showing 1 changed file with 0 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -536,17 +536,6 @@ public static Vector256<ushort> Load256(ref byte ptr)
[MethodImpl(MethodImplOptions.AggressiveInlining)]
public static Vector512<ushort> Load512(ref byte ptr)
{
// This is done here for performance gain.
// A similar implementation would be as below:
//
// (Vector256<ushort> lower, Vector256<ushort> upper) = Vector256.Widen(Vector256.LoadUnsafe(ref ptr));
// return Vector512.Create(lower, upper);
//
// This is similar to what is done for Load256 here. But
// for Vector512, this implementation is low performance
// since a load and widen on Vector256 followed by a
// create on Vector512 is leading to a performance lower
// than that of similar implementationfor Vector256.
(Vector512<ushort> lower, Vector512<ushort> _) = Vector512.Widen(Vector256.LoadUnsafe(ref ptr).ToVector512());
return lower;
}
Expand Down

0 comments on commit a995aae

Please sign in to comment.