Skip to content

Commit

Permalink
Remove all fields from SourceOrdinaryMethodSymbol in the common case. (
Browse files Browse the repository at this point in the history
…#68135)

* Use a singleton for empty type-parameters for named types

* move type parameters down

* Optimize methods as well

* Always create indirection

* In progresS

* Utilize

* initialize once

* In progresS

* Utilize

* initialize once

* Save space in other methods

* Move type

* reorder

* reorder

* Move down

* Reorder

* Simplify

* Make private

* lint

* Update src/Compilers/CSharp/Portable/Symbols/Source/SourceMemberMethodSymbol.cs

* Apply suggestions from code review

* Add docs

* Move

* Make sealed

* Update src/Compilers/CSharp/Portable/Symbols/Source/SourceOrdinaryMethodSymbol.cs

* Make abstract

* Better

* Update src/Compilers/CSharp/Portable/Symbols/Source/SourceOrdinaryMethodSymbol.cs

* Update src/Compilers/CSharp/Portable/Symbols/Source/SourceOrdinaryMethodSymbol.cs

* Update src/Compilers/CSharp/Portable/Symbols/Source/SourceOrdinaryMethodSymbol.cs

* Add comment

* Rename

* move

* Cleanup

* Comment

* MOve check

* Add back optimization

* use IsAssignableFrom

* Update src/Compilers/CSharp/Portable/Symbols/Source/SourceOrdinaryMethodSymbol.cs

Co-authored-by: Fred Silberberg <fred@silberberg.xyz>

* Update src/Compilers/CSharp/Portable/Symbols/Source/SourceOrdinaryMethodSymbol.cs

Co-authored-by: Fred Silberberg <fred@silberberg.xyz>

---------

Co-authored-by: Fred Silberberg <fred@silberberg.xyz>
  • Loading branch information
CyrusNajmabadi and 333fred authored May 18, 2023
1 parent 443c06d commit 3ca55ef
Show file tree
Hide file tree
Showing 2 changed files with 295 additions and 210 deletions.
Loading

0 comments on commit 3ca55ef

Please sign in to comment.