[PATCH] D53066: [Driver] Use forward slashes in most linker arguments

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 23 15:02:19 PDT 2018


rnk added inline comments.


================
Comment at: lib/Driver/Driver.cpp:1013-1014
   }
+  for (auto *Str : {&Dir, &InstalledDir, &SysRoot, &DyldPrefix, &ResourceDir})
+    *Str = llvm::sys::path::convert_to_slash(*Str);
 
----------------
zturner wrote:
> Is this going to affect the cl driver as well?
Yeah, if we change the resource dir, I think it's going to have knock-on affects to the debug info we emit to describe the locations of compiler intrinsic headers. I was imagining that this would be limited to flags which only get passed to GNU-ld-compatible linkers.


https://reviews.llvm.org/D53066





More information about the cfe-commits mailing list