nickdesaulniers added a comment.

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



================
Comment at: clang/test/Parser/asm-goto-with-outputs.c:7
+#error Extension 'gnu_asm_goto_with_outputs' should be available by default
+#endif
+
----------------
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?


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