This revision was automatically updated to reflect the committed changes. Closed by commit rC331666: [Driver] Use -fuse-line-directives by default in MSVC mode (authored by mstorsjo, committed by ).
Changed prior to commit: https://reviews.llvm.org/D46520?vs=145446&id=145528#toc Repository: rC Clang https://reviews.llvm.org/D46520 Files: lib/Driver/ToolChains/Clang.cpp test/Driver/cl-options.c Index: lib/Driver/ToolChains/Clang.cpp =================================================================== --- lib/Driver/ToolChains/Clang.cpp +++ lib/Driver/ToolChains/Clang.cpp @@ -4215,9 +4215,9 @@ IsWindowsMSVC)) CmdArgs.push_back("-fms-extensions"); - // -fno-use-line-directives is default. + // -fno-use-line-directives is default, except for MSVC targets. if (Args.hasFlag(options::OPT_fuse_line_directives, - options::OPT_fno_use_line_directives, false)) + options::OPT_fno_use_line_directives, IsWindowsMSVC)) CmdArgs.push_back("-fuse-line-directives"); // -fms-compatibility=0 is default. Index: test/Driver/cl-options.c =================================================================== --- test/Driver/cl-options.c +++ test/Driver/cl-options.c @@ -28,6 +28,7 @@ // RUN: %clang_cl /E -### -- %s 2>&1 | FileCheck -check-prefix=E %s // E: "-E" +// E: "-fuse-line-directives" // E: "-o" "-" // RUN: %clang_cl /EP -### -- %s 2>&1 | FileCheck -check-prefix=EP %s
Index: lib/Driver/ToolChains/Clang.cpp =================================================================== --- lib/Driver/ToolChains/Clang.cpp +++ lib/Driver/ToolChains/Clang.cpp @@ -4215,9 +4215,9 @@ IsWindowsMSVC)) CmdArgs.push_back("-fms-extensions"); - // -fno-use-line-directives is default. + // -fno-use-line-directives is default, except for MSVC targets. if (Args.hasFlag(options::OPT_fuse_line_directives, - options::OPT_fno_use_line_directives, false)) + options::OPT_fno_use_line_directives, IsWindowsMSVC)) CmdArgs.push_back("-fuse-line-directives"); // -fms-compatibility=0 is default. Index: test/Driver/cl-options.c =================================================================== --- test/Driver/cl-options.c +++ test/Driver/cl-options.c @@ -28,6 +28,7 @@ // RUN: %clang_cl /E -### -- %s 2>&1 | FileCheck -check-prefix=E %s // E: "-E" +// E: "-fuse-line-directives" // E: "-o" "-" // RUN: %clang_cl /EP -### -- %s 2>&1 | FileCheck -check-prefix=EP %s
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits