[clang] [clang][modules] Print library module manifest path. (PR #76451)

Chuanqi Xu via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 17 00:01:18 PST 2024


ChuanqiXu9 wrote:

> > LGTM.
> > We need to delete `clang/test/Driver/Inputs/cxx23_modules/usr/lib/x86_64-linux-gnu/libc++.so` and `clang/test/Driver/Inputs/cxx23_modules/usr/lib/x86_64-linux-gnu/modules.json`, we should generate them with `split-file`
> 
> Are you sure that is the proper way? Should these tests not use a complete installation to make sure it works on installations?

We can use `mkdir` to generate more complex paths. For example,

```
// RUN: split-file %s %t
//
// RUN: mkdir %t/....
// RUN: touch %t/.../libc++.so
```


The intention is to remove the use of `Inputs` in test. It is really hard to read.

https://github.com/llvm/llvm-project/pull/76451


More information about the cfe-commits mailing list