[PATCH] D79265: Let normalize() for posix style convert backslash to slash unconditionally.

Saleem Abdulrasool via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 4 14:32:22 PDT 2020


compnerd added a comment.

What happens if you encounter a `"\t"` as a string?  This would convert that to a `"/t"` would it not?  Although, I suppose that in practice the treatment of escaped characters is not important.  I think I still prefer the `} else {` here over the early return.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79265/new/

https://reviews.llvm.org/D79265





More information about the llvm-commits mailing list