Skip to content

Conversation

@AndyAyersMS
Copy link
Member

Summary

Update summary for MethodImplOptions.AggressiveOptimization to offer a bit more guidance on what it does and when it should be (or shouldn't be) used.

See dotnet/runtime#62148

cc @azhmur @EgorBo

@AndyAyersMS AndyAyersMS requested a review from a team as a code owner April 29, 2022 22:26
@opbld30
Copy link

opbld30 commented Apr 29, 2022

Docs Build status updates of commit 4071871:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Runtime.CompilerServices/MethodImplOptions.xml ✅Succeeded View

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

Copy link
Contributor

@gewarren gewarren left a comment

Choose a reason for hiding this comment

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

I split it into three paragraphs. What do you think?

@opbld30
Copy link

opbld30 commented Apr 30, 2022

Docs Build status updates of commit 50c9f95:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Runtime.CompilerServices/MethodImplOptions.xml ✅Succeeded View

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@gewarren
Copy link
Contributor

@azhmur
Copy link

azhmur commented Apr 30, 2022

Thank you. Looks good.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants