Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Cleanup some build files#24539

Merged
AaronRobinsonMSFT merged 5 commits into
masterfrom
cleanup_build_files
May 13, 2019
Merged

Cleanup some build files#24539
AaronRobinsonMSFT merged 5 commits into
masterfrom
cleanup_build_files

Conversation

@AaronRobinsonMSFT
Copy link
Copy Markdown
Member

No description provided.

@AaronRobinsonMSFT AaronRobinsonMSFT added area-Build * NO MERGE * The PR is not ready for merge yet (see discussion for detailed reasons) labels May 12, 2019
@@ -4,16 +4,24 @@

using System;
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dotnet/jit-contrib Can I get someone from the JIT team to sign off on this altered test? It has been timing out on ARM64-checked build. I attempted to reduce the runtime duration without changing the spirit of the test too much.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think #24174 should be reverted with this test change.

@AaronRobinsonMSFT AaronRobinsonMSFT removed the * NO MERGE * The PR is not ready for merge yet (see discussion for detailed reasons) label May 12, 2019
Copy link
Copy Markdown

@jashook jashook left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm thanks for the change

Comment thread src/AUTOEXP.DAT
@@ -1,2066 +0,0 @@
;---------------------------------------------------------------
; This comment block added by the CLR dev team.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

End of an era..do these not work anymore?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don’t know if this doesn’t work anymore, but Visual Studio has a new way of doing debug visualizers with natvis files so that we don’t have to change our VS installation. And WinDbg also supports them. https://docs.microsoft.com/en-us/visualstudio/debugger/create-custom-views-of-native-objects?view=vs-2019

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@danmosemsft These files are still supported, but .natvis files are now preferred. The issue with the AUTOEXP.DAT and native visualizers in general is that they are particularly sensitive to type layouts. As soon as one aspect of the type visualizer layout changes other aspects of the type visualizer can fail and AUTOEXP.DAT have a particularly poor diagnostic experience.

@AaronRobinsonMSFT
Copy link
Copy Markdown
Member Author

Timeout failures for some tests - non-obvious functional failures.

@AaronRobinsonMSFT AaronRobinsonMSFT merged commit 42a4b41 into master May 13, 2019
@AaronRobinsonMSFT AaronRobinsonMSFT deleted the cleanup_build_files branch May 13, 2019 00:23
franksinankaya pushed a commit to franksinankaya/coreclr that referenced this pull request May 30, 2019
* Remove OsEnvironment, PackagesDir, and CoreCLRBinDir properties
* Replace usage of __BuildArch with BuildArch
* Simplify test timing out on ARM64
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
* Remove OsEnvironment, PackagesDir, and CoreCLRBinDir properties
* Replace usage of __BuildArch with BuildArch
* Simplify test timing out on ARM64


Commit migrated from dotnet/coreclr@42a4b41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants