dotnet/runtime

[ILVerify]Add '--tokens' switch

オープン

#37,395 opened on 2018/12/18

 (0 件のコメント) (0 件のリアクション) (0 人の担当者)C# (5,445 件のフォーク)batch import
Priority:3area-Tools-ILVerificationhelp wanted

Repository metrics

Stars
 (17,886 個のスター)
PR merge metrics
 (PR metrics pending)

説明

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

コントリビューターガイド