[PATCH] D23816: clang-cl: Add support for /FC: absolute paths in diagnostics and __FILE__

Nico Weber via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 25 12:11:58 PDT 2016


thakis added inline comments.

================
Comment at: include/clang/Driver/CLCompatOptions.td:68
@@ +67,3 @@
+def _SLASH_FC : CLFlag<"FC">,
+  HelpText<"Use absolute paths in diagnostics and __FILE__">,
+  Alias<fabsolute_paths>;
----------------
Hm, I'm not sure this should affect __FILE__. It sounds people want this for diagnostics, but making it affect __FILE__ makes reproducible builds harder. (There's -ffile-macro-prefix-to-remove from http://reviews.llvm.org/D17741 for that too, but that's not in.)


https://reviews.llvm.org/D23816





More information about the cfe-commits mailing list