Remove internal binder flags (dotnet/coreclr#27476)
authorVitek Karas <vitek.karas@microsoft.com>
Sun, 27 Oct 2019 06:53:49 +0000 (23:53 -0700)
committerGitHub <noreply@github.com>
Sun, 27 Oct 2019 06:53:49 +0000 (23:53 -0700)
commit9da711ed9280b1f1c3f882b04c2a5e6b6ee26a74
treef44dd52dd454c31953f80450845c7dfc8602dc39
parenteb17cdc82f9ace479bf8713976125c9e70866d77
Remove internal binder flags (dotnet/coreclr#27476)

Removed flags:
* Dynamic bind flags were only used for one assert - which is obviously always true (otherwise it would fail)
* Ignore dynamic bind flag was never used.
* Bind cache failures flags was on the other hand always used - so hardcode it directly
* Change the other flags to simple booleans as it makes the code easier to read (and there are now only 2 such flags). Also renamed:
  * cache rerun -> skip failure caching
  * ignore ref/def match -> skip version compatibility check

Minor additional cleanup

Commit migrated from https://github.com/dotnet/coreclr/commit/7c5a46c27f2120528b13e7db78521134531e5edc
src/coreclr/src/binder/assemblybinder.cpp
src/coreclr/src/binder/clrprivbindercoreclr.cpp
src/coreclr/src/binder/inc/assembly.hpp
src/coreclr/src/binder/inc/assembly.inl
src/coreclr/src/binder/inc/assemblybinder.hpp
src/coreclr/src/binder/inc/bindresult.hpp
src/coreclr/src/binder/inc/bindresult.inl
src/coreclr/src/binder/inc/contextentry.hpp
src/coreclr/src/binder/inc/loadcontext.inl
src/coreclr/src/vm/appdomain.cpp