-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Conversation
@@ -16,6 +18,10 @@ The implementation of this API was making some questionable assumptions about Wi | |||
|
|||
See github issue [#13153](https://github.com/dotnet/coreclr/issues/13153) for more details. | |||
|
|||
### ReJIT on ARM | |||
|
|||
ReJIT optimizations are only available on x86/x64 for now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this mean that the feature works fine, but it has worse performance than on x86/x64?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh. I meant that ReJIT feature itself is not working on ARM. I'll change the word "optimizations" to "feature" to avoid ambiguity. Thank you!
@@ -16,6 +18,10 @@ The implementation of this API was making some questionable assumptions about Wi | |||
|
|||
See github issue [#13153](https://github.com/dotnet/coreclr/issues/13153) for more details. | |||
|
|||
### ReJIT on ARM | |||
|
|||
ReJIT feature is only available on x86/x64 for now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets open an issue that tracks this and link it here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it. Thanks!
LGTM, modulo my one comment |
Platform testing for Windows/Linux on arm32 as well as OSX have been completed. Updating the documentation to reflect this change.