[PATCH] D135818: [clang] Update ASM goto documentation to reflect how Clang differs from GCC

2022-10-13 Thread Bill Wendling via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. void marked an inline comment as done. Closed by commit rG8c7b3461a534: [clang] Update ASM goto documentation to reflect how Clang differs from GCC (authored by void). Changed prior to commit: https://reviews.llvm.org/D13

[PATCH] D135818: [clang] Update ASM goto documentation to reflect how Clang differs from GCC

2022-10-13 Thread Bill Wendling via Phabricator via cfe-commits
void marked 2 inline comments as done. void added inline comments. Comment at: clang/docs/LanguageExtensions.rst:1553 + implementation`_ in + that Clang doesn't support outputs on the indirect branch. Use of an output + on

[PATCH] D135818: [clang] Update ASM goto documentation to reflect how Clang differs from GCC

2022-10-13 Thread Shafik Yaghmour via Phabricator via cfe-commits
shafik added inline comments. Comment at: clang/docs/LanguageExtensions.rst:1555 + on the indirect branch may result in undefined behavior and should be + avoided. void wrote: > shafik wrote: > > It would be helpful to provide an example of what is not suppor

[PATCH] D135818: [clang] Update ASM goto documentation to reflect how Clang differs from GCC

2022-10-13 Thread Nick Desaulniers via Phabricator via cfe-commits
nickdesaulniers accepted this revision. nickdesaulniers added inline comments. This revision is now accepted and ready to land. Comment at: clang/docs/LanguageExtensions.rst:1553 + implementation`_ in + that Clang doesn't su

[PATCH] D135818: [clang] Update ASM goto documentation to reflect how Clang differs from GCC

2022-10-12 Thread Bill Wendling via Phabricator via cfe-commits
void added inline comments. Comment at: clang/docs/LanguageExtensions.rst:1555 + on the indirect branch may result in undefined behavior and should be + avoided. shafik wrote: > It would be helpful to provide an example of what is not supported. If you > can

[PATCH] D135818: [clang] Update ASM goto documentation to reflect how Clang differs from GCC

2022-10-12 Thread Bill Wendling via Phabricator via cfe-commits
void updated this revision to Diff 467284. void added a comment. Update with example. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D135818/new/ https://reviews.llvm.org/D135818 Files: clang/docs/LanguageExtensions.rst Index: clang/docs/Languag

[PATCH] D135818: [clang] Update ASM goto documentation to reflect how Clang differs from GCC

2022-10-12 Thread Shafik Yaghmour via Phabricator via cfe-commits
shafik added a comment. Thank you for updating the documents! Comment at: clang/docs/LanguageExtensions.rst:1555 + on the indirect branch may result in undefined behavior and should be + avoided. It would be helpful to provide an example of what is not supp

[PATCH] D135818: [clang] Update ASM goto documentation to reflect how Clang differs from GCC

2022-10-12 Thread Bill Wendling via Phabricator via cfe-commits
void created this revision. void added a reviewer: nickdesaulniers. Herald added a project: All. void requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D135818 Files: clang/do