[PATCH] D112289: Support: Use sys::path::system_style() in a few places

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 21 18:38:09 PDT 2021


dexonsmith created this revision.
dexonsmith added a reviewer: Bigcheese.
Herald added a subscriber: hiraditya.
dexonsmith requested review of this revision.
Herald added projects: clang, LLVM.

Use the new sys::path::system_style() in a few places that need to
detect the system's native path style, removing a few `_WIN32` macro
checks. Added one FIXME to a FileManagerTest that seemed fishy, but
maintained the existing behaviour.

Happy to split this up as much as makes sense. Mainly posting to show examples for https://reviews.llvm.org/D112288.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D112289

Files:
  clang/include/clang/Basic/JsonSupport.h
  clang/lib/Basic/FileManager.cpp
  clang/lib/Driver/Driver.cpp
  clang/lib/Driver/ToolChain.cpp
  clang/lib/Lex/PPDirectives.cpp
  clang/unittests/Basic/FileManagerTest.cpp
  clang/unittests/Driver/ToolChainTest.cpp
  clang/unittests/Lex/HeaderSearchTest.cpp
  clang/unittests/Tooling/RefactoringTest.cpp
  llvm/include/llvm/Support/VirtualFileSystem.h
  llvm/lib/Support/GraphWriter.cpp
  llvm/tools/lli/lli.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D112289.381456.patch
Type: text/x-patch
Size: 16942 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20211022/c795e839/attachment.bin>


More information about the cfe-commits mailing list