Skip to content

Add reflection introspection support for function pointers #81006

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
steveharter merged 24 commits into from
Feb 16, 2023
Merged

Add reflection introspection support for function pointers #81006

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
173b84d
Add reflection introspection support for function pointers
steveharter May 31, 2022
e546ee2
Fix compiler issue; test MLC to NetFramework
steveharter Jan 23, 2023
36e07d6
Fix compile issue; prep for review
steveharter Jan 23, 2023
e755c1c
Compile issue with MLC under certain conditions
steveharter Jan 23, 2023
b3b64ff
compile issue cont'd; lazy load mods on root
steveharter Jan 23, 2023
3aef789
compile issues cont'd; prepare for review
steveharter Jan 24, 2023
f76c75d
Remove Node back reference; fix edge case test failure
steveharter Jan 24, 2023
73b15b3
Increase test coverage; add missing recursive check
steveharter Jan 24, 2023
1361142
Various feedback; support mods on generic type parameters
steveharter Jan 26, 2023
520a3f5
Merge branch 'main' of https://github.com/steveharter/runtime into Fc…
steveharter Jan 26, 2023
64a987e
Fix MLC edge case; Native AOT test enable\disable
steveharter Jan 26, 2023
5f8a81e
Native AOT test enable\disable; add a generic test
steveharter Jan 26, 2023
2df9d2b
Native AOT test enable\disable; fix missing overload
steveharter Jan 26, 2023
0a6ff32
Add TODOs based on review discussion; MLC field optimization
steveharter Jan 27, 2023
784df92
Merge branch 'main' into FcnPtr
jkotas Feb 1, 2023
8c06d3c
Replace nested signature indices by TypeSignature type
jkotas Jan 31, 2023
f40e506
Move tests into separate classes; other misc non-functional
steveharter Feb 13, 2023
4610e93
Throw NSE for modified type members than may return an unmodified type
steveharter Feb 14, 2023
ca45bd8
Merge branch 'main' of https://github.com/dotnet/runtime into FcnPtr
steveharter Feb 14, 2023
70529f0
Throw NSE on modified type Equals() and GetHashCode()
steveharter Feb 15, 2023
9770a2d
Fix tests for WASI
steveharter Feb 15, 2023
14a9007
Remove unnecessary high overhead tests
steveharter Feb 16, 2023
69ccd75
Merge branch 'main' of https://github.com/dotnet/runtime into FcnPtr
steveharter Feb 16, 2023
f76331c
Fix merge error
steveharter Feb 16, 2023
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
Increase test coverage; add missing recursive check
  • Loading branch information
@steveharter
steveharter committed Jan 24, 2023
commit 73b15b3c4395ecb9bc4f2e5f67cdad85539ae35e
5 changes: 2 additions & 3 deletions src/coreclr/utilcode/sigparser.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -301,12 +301,13 @@ HRESULT SigParser::MoveToSignature(uint32_t indexToFind, uint32_t currentIndex,

// Handle return type
IfFailRet(MoveToSignature(indexToFind, currentIndex, isFinished));
if (*isFinished)
return S_OK;

// Handle args
while (fpArgCnt--)
{
IfFailRet(MoveToSignature(indexToFind, currentIndex, isFinished));

if (*isFinished)
return S_OK;
}
Expand Down Expand Up @@ -337,7 +338,6 @@ HRESULT SigParser::MoveToSignature(uint32_t indexToFind, uint32_t currentIndex,
IfFailRet(GetData(NULL)); // Skip lower bounds
}
}

}
break;

Expand All @@ -359,7 +359,6 @@ HRESULT SigParser::MoveToSignature(uint32_t indexToFind, uint32_t currentIndex,
while (argCnt--)
{
IfFailRet(MoveToSignature(indexToFind, currentIndex, isFinished));

if (*isFinished)
return S_OK;
}
Expand Down
Loading