Re: [PATCH] D15926: Do not print certain warnings when input is a header.

2016-07-05 Thread Mehdi AMINI via cfe-commits
mehdi_amini added inline comments. Comment at: include/clang/Basic/LangOptions.h:131 @@ +130,3 @@ + /// input is a header file (i.e. -x c-header). + bool IsHeaderFile = false; + aaron.ballman wrote: > Should move the initializer to the constructor (not certain

Re: [PATCH] D15926: Do not print certain warnings when input is a header.

2016-07-05 Thread Aaron Ballman via cfe-commits
aaron.ballman added a subscriber: aaron.ballman. Comment at: include/clang/Basic/LangOptions.h:131 @@ +130,3 @@ + /// input is a header file (i.e. -x c-header). + bool IsHeaderFile = false; + Should move the initializer to the constructor (not certain that MSVC

Re: [PATCH] D15926: Do not print certain warnings when input is a header.

2016-07-05 Thread Erik Verbruggen via cfe-commits
erikjv added a comment. ping? http://reviews.llvm.org/D15926 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D15926: Do not print certain warnings when input is a header.

2016-06-08 Thread Erik Verbruggen via cfe-commits
erikjv updated this revision to Diff 60023. http://reviews.llvm.org/D15926 Files: include/clang/Basic/LangOptions.h lib/Frontend/CompilerInvocation.cpp lib/Lex/PPDirectives.cpp lib/Lex/PPMacroExpansion.cpp lib/Lex/Pragma.cpp test/Preprocessor/header_is_main_file.c Index:

[PATCH] D15926: Do not print certain warnings when input is a header.

2016-01-06 Thread Erik Verbruggen via cfe-commits
erikjv created this revision. erikjv added reviewers: klimek, rsmith. erikjv added a subscriber: cfe-commits. When generating pre-compiled headers, or when opening a header file with libclang, suppress the warnings "#pragma once in main file" and "#include_next in primary source file". Fixes

Re: [PATCH] D15926: Do not print certain warnings when input is a header.

2016-01-06 Thread Mehdi AMINI via cfe-commits
joker.eph added a subscriber: joker.eph. Comment at: include/clang/Basic/SourceManager.h:660 @@ -659,1 +659,3 @@ + bool MainFileIsHeader = false; + Document? Comment at: include/clang/Basic/SourceManager.h:764 @@ -761,1 +763,3 @@ + void