[PATCH] D88721: [clangd][lit] Update document-link.test to respect custom resource-dir locations

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 2 04:28:50 PDT 2020


sammccall added a comment.

> As far as Windows accepts forward and back slashes

Note we don't rely on windows itself for this support.
All access through `llvm::sys::fs` APIs on windows ultimately goes through `widenPath` to convert to UTF-16, and this substitutes slashes.
So LLVM tools do always support `/` on windows and it's fairly common to rely on this for tests (abstraction is hard in lit tests).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88721



More information about the cfe-commits mailing list