ken-matsui added a comment. @aaron.ballman
Thank you for your review and sorry to have missed those directives! I also found additional directives that I might have to diagnose. Could you please tell me if these are also required? ================ Comment at: clang/test/Preprocessor/line-directive.c:67-77 #line 42 "blonk.h" // doesn't change system headerness. typedef int z; // ok typedef int z; // ok # 97 // doesn't change system headerness. ---------------- Should these directives also be diagnosed? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124534/new/ https://reviews.llvm.org/D124534 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
