[PATCH] D73319: Add extension 'gnu_asm_goto_with_outputs'

Bill Wendling via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 27 13:11:31 PST 2020


void added a comment.

In D73319#1842900 <https://reviews.llvm.org/D73319#1842900>, @nickdesaulniers wrote:

> Great idea; thanks for the patch! Needs documentation. :)


I can't find a place for documentation of extensions, at least not the names of extensions like `gnu_asm`.



================
Comment at: clang/test/Parser/asm-goto-with-outputs.c:7
+#error Extension 'gnu_asm_goto_with_outputs' should be available by default
+#endif
+
----------------
nickdesaulniers wrote:
> probably don't need a bunch of test cases, just the check for the preprocessor define?  In that case, why don't you just add the above to an existing asm goto test?
Sure. done.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73319/new/

https://reviews.llvm.org/D73319





More information about the cfe-commits mailing list