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

Conversation

@franksinankaya
Copy link

@franksinankaya franksinankaya changed the title ARM compilation fixes with GCC ARM32 compilation fixes with GCC Nov 10, 2019
@jkotas jkotas added area-Build and removed area-VM labels Nov 11, 2019
@franksinankaya
Copy link
Author

any other comments?

@jashook
Copy link

jashook commented Nov 12, 2019

ping @janvorli

@franksinankaya
Copy link
Author

@janvorli : any other feedback on this PR?

Copy link
Member

@janvorli janvorli left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@janvorli janvorli merged commit e8bbcf1 into dotnet:master Nov 13, 2019
MichalStrehovsky pushed a commit to MichalStrehovsky/corert that referenced this pull request Mar 28, 2020
* Suppress on clang only

* Fix integer conversion

* Extra qualifier

* Suppress warning

* Extra qualifier

* Signedness issues

* Correct offsetof

* Offsetof doesn't support non-constant values

* Conversion errors

* Move the comment too

* Fix assembly warning

* size is not constant

* Fix comment type

* Fix endmacro name

* Use OFFSET_NONE constant

Commit migrated from dotnet/coreclr@e8bbcf1
MichalStrehovsky pushed a commit to MichalStrehovsky/corert that referenced this pull request Mar 31, 2020
* Suppress on clang only

* Fix integer conversion

* Extra qualifier

* Suppress warning

* Extra qualifier

* Signedness issues

* Correct offsetof

* Offsetof doesn't support non-constant values

* Conversion errors

* Move the comment too

* Fix assembly warning

* size is not constant

* Fix comment type

* Fix endmacro name

* Use OFFSET_NONE constant

Commit migrated from dotnet/coreclr@e8bbcf1
jkotas pushed a commit to dotnet/corert that referenced this pull request Apr 1, 2020
* Suppress on clang only

* Fix integer conversion

* Extra qualifier

* Suppress warning

* Extra qualifier

* Signedness issues

* Correct offsetof

* Offsetof doesn't support non-constant values

* Conversion errors

* Move the comment too

* Fix assembly warning

* size is not constant

* Fix comment type

* Fix endmacro name

* Use OFFSET_NONE constant

Commit migrated from dotnet/coreclr@e8bbcf1
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.

5 participants