[PATCH] D45942: [Support/Path] Make handling of paths like "///" consistent

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 23 09:19:23 PDT 2018


zturner added inline comments.


================
Comment at: unittests/Support/Path.cpp:186-190
+  EXPECT_EQ("/", path::filename("//"));
+  EXPECT_EQ("", path::parent_path("//"));
+
+  EXPECT_EQ("/", path::filename("///"));
+  EXPECT_EQ("", path::parent_path("///"));
----------------
A comment here maybe in order.  Should this behavior be true for both path styles?


Repository:
  rL LLVM

https://reviews.llvm.org/D45942





More information about the llvm-commits mailing list