Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
dotNET Platform
runtime
提交
753d12fa
R
runtime
项目概览
dotNET Platform
/
runtime
12 个月 前同步成功
通知
1
Star
1
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
DevOps
流水线
流水线任务
计划
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
R
runtime
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
DevOps
DevOps
流水线
流水线任务
计划
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
流水线任务
提交
Issue看板
体验新版 GitCode,发现更多精彩内容 >>
未验证
提交
753d12fa
编写于
8月 11, 2022
作者:
M
Michal Strehovský
提交者:
GitHub
8月 11, 2022
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
Ensure methods on different instantiations are reflectable (#73749)
上级
17154bd7
变更
2
隐藏空白更改
内联
并排
Showing
2 changed file
with
39 addition
and
2 deletion
+39
-2
src/coreclr/tools/aot/ILCompiler.Compiler/Compiler/UsageBasedMetadataManager.cs
...ILCompiler.Compiler/Compiler/UsageBasedMetadataManager.cs
+19
-2
src/tests/nativeaot/SmokeTests/Reflection/Reflection.cs
src/tests/nativeaot/SmokeTests/Reflection/Reflection.cs
+20
-0
未找到文件。
src/coreclr/tools/aot/ILCompiler.Compiler/Compiler/UsageBasedMetadataManager.cs
浏览文件 @
753d12fa
...
...
@@ -465,9 +465,9 @@ public override void GetConditionalDependenciesDueToEETypePresence(ref CombinedD
// ObjectGetTypeFlowDependencies don't need to be conditional in that case. They'll be added as needed.
}
// Ensure fields can be consistently reflection set & get.
if
(
type
.
HasInstantiation
&&
!
type
.
IsTypeDefinition
&&
!
IsReflectionBlocked
(
type
))
{
// Ensure fields can be consistently reflection set & get.
foreach
(
FieldDesc
field
in
type
.
GetFields
())
{
// Tiny optimization: no get/set for literal fields since they only exist in metadata
...
...
@@ -481,7 +481,24 @@ public override void GetConditionalDependenciesDueToEETypePresence(ref CombinedD
dependencies
.
Add
(
new
DependencyNodeCore
<
NodeFactory
>.
CombinedDependencyListEntry
(
factory
.
ReflectableField
(
field
),
factory
.
ReflectableField
(
field
.
GetTypicalFieldDefinition
()),
"GetType called on the interface"
));
"Fields have same reflectability"
));
}
// Ensure methods can be consistently reflection-accessed
foreach
(
MethodDesc
method
in
type
.
GetMethods
())
{
if
(
IsReflectionBlocked
(
method
))
continue
;
// Generic methods need to be instantiated over something.
if
(
method
.
HasInstantiation
)
continue
;
dependencies
??=
new
CombinedDependencyList
();
dependencies
.
Add
(
new
DependencyNodeCore
<
NodeFactory
>.
CombinedDependencyListEntry
(
factory
.
ReflectableMethod
(
method
),
factory
.
ReflectableMethod
(
method
.
GetTypicalMethodDefinition
()),
"Methods have same reflectability"
));
}
}
}
...
...
src/tests/nativeaot/SmokeTests/Reflection/Reflection.cs
浏览文件 @
753d12fa
...
...
@@ -62,6 +62,7 @@ private static int Main()
TestInvokeMethodMetadata
.
Run
();
TestVTableOfNullableUnderlyingTypes
.
Run
();
TestInterfaceLists
.
Run
();
TestMethodConsistency
.
Run
();
//
// Mostly functionality tests
...
...
@@ -2021,6 +2022,25 @@ public static void Run()
}
}
class
TestMethodConsistency
{
class
MyGenericType
<
T
>
{
public
static
string
MyMethod
()
=>
typeof
(
T
).
Name
;
}
struct
Atom
{
}
public
static
void
Run
()
{
object
returned
=
Grab
<
Atom
>().
Invoke
(
null
,
null
);
if
((
string
)
returned
!=
nameof
(
Atom
))
throw
new
Exception
();
static
MethodInfo
Grab
<
T
>()
=>
typeof
(
MyGenericType
<
T
>).
GetMethod
(
nameof
(
MyGenericType
<
T
>.
MyMethod
));
}
}
#
region
Helpers
private
static
Type
SecretGetType
(
string
testName
,
string
typeName
)
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录