Michael137 wrote: > Nice. Now using a simple test like > > ``` > // test.cpp > int __clang_vtable; > ``` > > and > > ``` > clang -c test.cpp -Wreserved-identifier > test.cpp:2:5: warning: identifier '__clang_vtable' is reserved because it > starts with '__' [-Wreserved-identifier] > 2 | int __clang_vtable; > | ^ > 1 warning generated. > ``` > > May be add a simple test case to indicate that now that name is a Clang > reserved identifier.
Not sure it's worth writing a test-case for that since it'd just be testing whether clang warns on `__` being reserved. For which a bunch of tests already presumably exist. Going to merge for now. Let me know if I misunderstood the test suggestion and I'll add it retroactively. https://github.com/llvm/llvm-project/pull/183617 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
