dotnet/runtime

[ILVerify]Add '--tokens' switch

开放

#37,395 创建于 2018年12月18日

 (0 条评论) (0 个反应) (0 位负责人)C# (5,445 个派生)batch import
Priority:3area-Tools-ILVerificationhelp wanted

仓库指标

星标
 (17,886 个星标)
PR 合并指标
 (PR 指标待抓取)

描述

Today when ILVerify found issue we report tokens/offsets value for type system item for IL methods i.e.

[IL]: Error: [C:\git\corert\bin\Windows_NT.x64.Debug\ILVerification.Tests\Tests\BranchingTests.dll : BranchingTestsType::Branching.FromTryIntoFinally_Invalid_BranchOutOfTry.BranchIntoHandler()][offset 0x00000004] Branch out of try block.

We added some new type check and we decided to not report token value for it because they are not useful for regular use i.e.

[MD]: Error: Interface implementation has a duplicate. Class '[InterfaceImplementation]InterfaceImplHasDuplicate_InvalidType_InterfaceImplHasDuplicate' Interface: '[InterfaceDefinition]Interface'

However there are some scenario where tokens are useful, for instance on obfuscated code. We should refactor code to have token values only if a user pass '--tokens' switch to command line. Reference https://github.com/dotnet/corert/pull/6687#discussion_r241916514

Task list:

Extra: cleanup code https://github.com/dotnet/corert/pull/6758#discussion_r248405930

贡献者指南