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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to