================
@@ -950,12 +957,9 @@ void ModuleDepCollector::addVisibleModules() {
static StringRef makeAbsoluteAndPreferred(CompilerInstance &CI, StringRef Path,
SmallVectorImpl<char> &Storage) {
- if (llvm::sys::path::is_absolute(Path) &&
- !llvm::sys::path::is_style_windows(llvm::sys::path::Style::native))
- return Path;
----------------
jansvoboda11 wrote:
This is just an optimization to not re allocate the path if it's absolute and
doesn't need separator canonicalization. Now that that I need to call
`remove_dots()`
(https://github.com/llvm/llvm-project/pull/182063#discussion_r2823411756), I'd
need to check that this path doesn't contain dot components to see if it's safe
to skip the code below. So I decided to just drop the optimization entirely.
https://github.com/llvm/llvm-project/pull/182063
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
