thakis created this revision. thakis added a reviewer: majnemer. thakis added a subscriber: cfe-commits.
/Brepro means we want reproducible builds, i.e. we _don't_ want the timestamp that's needed to be compatible with the incremental linker. https://reviews.llvm.org/D23805 Files: include/clang/Driver/CLCompatOptions.td test/Driver/cl-options.c Index: test/Driver/cl-options.c =================================================================== --- test/Driver/cl-options.c +++ test/Driver/cl-options.c @@ -432,10 +432,10 @@ // BreproDefault: "-mincremental-linker-compatible" // RUN: %clang_cl /Brepro- /Brepro /c '-###' -- %s 2>&1 | FileCheck -check-prefix=Brepro %s -// Brepro: "-mincremental-linker-compatible" +// Brepro-NOT: "-mincremental-linker-compatible" // RUN: %clang_cl /Brepro /Brepro- /c '-###' -- %s 2>&1 | FileCheck -check-prefix=Brepro_ %s -// Brepro_-NOT: "-mincremental-linker-compatible" +// Brepro_: "-mincremental-linker-compatible" // This test was super sneaky: "/Z7" means "line-tables", but "-gdwarf" occurs // later on the command line, so it should win. Interestingly the cc1 arguments Index: include/clang/Driver/CLCompatOptions.td =================================================================== --- include/clang/Driver/CLCompatOptions.td +++ include/clang/Driver/CLCompatOptions.td @@ -54,10 +54,10 @@ def _SLASH_Brepro : CLFlag<"Brepro">, HelpText<"Emit an object file which can be reproduced over time">, - Alias<mincremental_linker_compatible>; + Alias<mno_incremental_linker_compatible>; def _SLASH_Brepro_ : CLFlag<"Brepro-">, HelpText<"Emit an object file which cannot be reproduced over time">, - Alias<mno_incremental_linker_compatible>; + Alias<mincremental_linker_compatible>; def _SLASH_C : CLFlag<"C">, HelpText<"Don't discard comments when preprocessing">, Alias<C>; def _SLASH_c : CLFlag<"c">, HelpText<"Compile only">, Alias<c>;
Index: test/Driver/cl-options.c =================================================================== --- test/Driver/cl-options.c +++ test/Driver/cl-options.c @@ -432,10 +432,10 @@ // BreproDefault: "-mincremental-linker-compatible" // RUN: %clang_cl /Brepro- /Brepro /c '-###' -- %s 2>&1 | FileCheck -check-prefix=Brepro %s -// Brepro: "-mincremental-linker-compatible" +// Brepro-NOT: "-mincremental-linker-compatible" // RUN: %clang_cl /Brepro /Brepro- /c '-###' -- %s 2>&1 | FileCheck -check-prefix=Brepro_ %s -// Brepro_-NOT: "-mincremental-linker-compatible" +// Brepro_: "-mincremental-linker-compatible" // This test was super sneaky: "/Z7" means "line-tables", but "-gdwarf" occurs // later on the command line, so it should win. Interestingly the cc1 arguments Index: include/clang/Driver/CLCompatOptions.td =================================================================== --- include/clang/Driver/CLCompatOptions.td +++ include/clang/Driver/CLCompatOptions.td @@ -54,10 +54,10 @@ def _SLASH_Brepro : CLFlag<"Brepro">, HelpText<"Emit an object file which can be reproduced over time">, - Alias<mincremental_linker_compatible>; + Alias<mno_incremental_linker_compatible>; def _SLASH_Brepro_ : CLFlag<"Brepro-">, HelpText<"Emit an object file which cannot be reproduced over time">, - Alias<mno_incremental_linker_compatible>; + Alias<mincremental_linker_compatible>; def _SLASH_C : CLFlag<"C">, HelpText<"Don't discard comments when preprocessing">, Alias<C>; def _SLASH_c : CLFlag<"c">, HelpText<"Compile only">, Alias<c>;
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
