Skip to content
This repository was archived by the owner on Nov 1, 2020. It is now read-only.

Add tests for static field layout#33

Merged
MichalStrehovsky merged 1 commit into
dotnet:masterfrom
MichalStrehovsky:StaticTests
Oct 7, 2015
Merged

Add tests for static field layout#33
MichalStrehovsky merged 1 commit into
dotnet:masterfrom
MichalStrehovsky:StaticTests

Conversation

@MichalStrehovsky
Copy link
Copy Markdown
Member

Porting native NUTC tests for static field layout to the new type
system.

Porting native NUTC tests for static field layout to the new type
system.
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.

Should we detect the architecture from the environment as opposed to hard-coding it?

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.

That would mean we can only test say - field layout for ARM64 - only when the test runs on an ARM64 device.

We should test other platforms, but we can run all the tests on all platforms irrespective of the current platform.

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.

If X64 is translated as "any 64bit" target, then the above works. However, what happens for 32bit (x86 vs Arm32)? If the target architecture is passed to the test infra at command-line (or such mechanism) that allows the architecture to be not hard-coded, your goal of running arm64 validations on x64 should still work.

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.

x64 is exactly AMD64 (or x86-64). Other 64 bit architectures might have e.g. different requirements for packing or alignments. I don't know and the type system doesn't know either (because all I implemented is x64).

@jkotas
Copy link
Copy Markdown
Member

jkotas commented Oct 7, 2015

LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants