Skip to content

Add mention of C++ AMP deprecation#3221

Merged
TylerMSFT merged 5 commits into
MicrosoftDocs:masterfrom
barcharcraz:amp_deprecate
Jun 29, 2021
Merged

Add mention of C++ AMP deprecation#3221
TylerMSFT merged 5 commits into
MicrosoftDocs:masterfrom
barcharcraz:amp_deprecate

Conversation

@barcharcraz

Copy link
Copy Markdown

C++ AMP headers are deprecated in dev17, add note mentioning that and how to suppress the deprecation error.

I believe the "whats-new-cpp-docs" page I updated is the correct one, but it mentions VS 16.8, so I'm not totally sure.

@PRMerger9

Copy link
Copy Markdown
Contributor

@barcharcraz : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@PRMerger16

Copy link
Copy Markdown
Contributor

@barcharcraz : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Comment thread docs/parallel/amp/cpp-amp-overview.md Outdated
@PRMerger6 PRMerger6 requested a review from TylerMSFT June 26, 2021 00:15
@PRMerger6

Copy link
Copy Markdown
Contributor

@barcharcraz : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Comment thread docs/overview/whats-new-cpp-docs.md Outdated
- [`assert Macro, _assert, _wassert`](../c-runtime-library/reference/assert-macro-assert-wassert.md) - Clarified assert behavior
- [`vsnprintf_s, _vsnprintf_s, _vsnprintf_s_l, _vsnwprintf_s, _vsnwprintf_s_l`](../c-runtime-library/reference/vsnprintf-s-vsnprintf-s-vsnprintf-s-l-vsnwprintf-s-vsnwprintf-s-l.md) - Clarified return values
- [`setlocale, _wsetlocale`](../c-runtime-library/reference/setlocale-wsetlocale.md) - Added C Runtime UTF-8 support info
- [`C++ AMP Overview`](../parallel/amp/cpp-amp-overview.md) - Added note about C++ AMP deprecation.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

What do you think about pulling this one change out. It's sitting in the section for what's new in VS 16.8
I won't create a new section for VS 2022 until VS 2022 goes out (otherwise it looks like VS 2022 is released).

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

sure, but we need to remember to add the note to the release notes. It's important people know about the change when upgrading.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

These aren't the release notes. This should be noted in the release notes, but not here (yet).

@PRMerger6 PRMerger6 requested a review from TylerMSFT June 29, 2021 19:57
@PRMerger6

Copy link
Copy Markdown
Contributor

@barcharcraz : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@TylerMSFT TylerMSFT left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks, @barcharcraz.

@TylerMSFT TylerMSFT merged commit 3cc4398 into MicrosoftDocs:master Jun 29, 2021
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.

7 participants