[clang] Reland Print library module manifest path again (PR #84881)

Chuanqi Xu via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 18 06:08:45 PDT 2024


ChuanqiXu9 wrote:

> > > The file we're looking for is `modules.json`; Renaming it `libc++.modules.json` like `.so` / `.a` file might be a better idea which could avoid name clashes when installed in `/usr/lib`.
> > 
> > 
> > but i didn't rename it, it was with the libc++ prefix directly :/
> 
> Good point I did :-/ It seems we originally talked about `modules.json` and later mentioned using the libname. @ChuanqiXu9 I prefer to keep the current name `libc++.modules.json` and adjust clang. WDYT?

Could you elaborate the reason more? I feel like renaming `libc++.modules.json` to `modules.json` is straightforward. Maybe I didn't take part it in the old disscussion or I missed : )

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


More information about the cfe-commits mailing list