[PATCH] D28444: Define sys::path::convert_to_slash.

Rafael Avila de Espindola via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 8 07:51:17 PST 2017


Rui Ueyama via Phabricator <reviews at reviews.llvm.org> writes:
> +/// @brief Replaces backslashes with slashes if Windows.
> +///
> +/// @param path processed path
> +/// @result The result of replacing backslashes with forward slashes if Windows.
> +/// On Unix, this function is a no-op.
> +std::string convert_to_slash(StringRef path);

Please document why it is a nop on Unix (\ is a valid filename
character).

Cheers,
Rafael


More information about the llvm-commits mailing list