mizvekov added inline comments.

================
Comment at: clang/test/Parser/while-loop-outside-function.c:3-4
+
+while(1) {}; // expected-error {{expected identifier or '('}} \
+             // expected-note {{while loop outside of functions}}
+
----------------
inclyc wrote:
> mizvekov wrote:
> > How about just making the error be the same thing the note would say?
> > 
> > 
> OK, so just create a new error directly, and then report the error when 
> encountering the `while` token?
Well the end result should be that there is only one error, `while` outside of 
function, because that explains everything the user needs to know.

It would be strange to report the `expected identifier or ...` error in 
addition to this one, it would be just noise.

How to go about implementing that I leave it up to you :P
Unless you need some specific help of course.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129573

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

Reply via email to