[PATCH] D49466: Initial implementation of -fmacro-prefix-map and -ffile-prefix-map

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 26 20:32:56 PST 2019


MaskRay added a comment.

The ugly path separator pattern `{{(/|\\\\)}}` appears in 60+ tests. Can we teach clang and other tools to

1. accept both `/` and `\` input
2. but only output `/`

on Windows? We can probably remove `llvm::sys::path::Style::{windows,posix,native}` from `include/llvm/Support/Path.h` and only keep the posix form.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D49466





More information about the llvm-commits mailing list