search

llvm / llvm-project

The LLVM Project is a collection of modular and reusable compiler and toolchain technologies.
http://llvm.org
Other
27.3k stars 11.19k forks source link

[C++20] [Modules] `-print-library-module-manifest-path` is broken now #97025

Open ChuanqiXu9 opened 1 month ago

ChuanqiXu9 commented 1 month ago

See https://github.com/llvm/llvm-project/issues/73089#issuecomment-2196357282

After the modules are installed properly, we can find the manifest file with -print-file-name but not with -print-library-module-manifest-path. This is not intentional and we should fix this.

llvmbot commented 1 month ago

@llvm/issue-subscribers-clang-modules

Author: Chuanqi Xu (ChuanqiXu9)

See https://github.com/llvm/llvm-project/issues/73089#issuecomment-2196357282 After the modules are installed properly, we can find the manifest file with `-print-file-name` but not with `-print-library-module-manifest-path`. This is not intentional and we should fix this.
mordante commented 1 month ago

Thanks for creating the bug. Do you have time to look into this before the LLVM-19 release?

ChuanqiXu9 commented 4 weeks ago

Thanks for creating the bug. Do you have time to look into this before the LLVM-19 release?

I may take a try but I can't make a promise.

ChuanqiXu9 commented 3 weeks ago

Thanks for creating the bug. Do you have time to look into this before the LLVM-19 release?

hmm it looks like I can't reproduce this. Did you reproduce this before?

mordante commented 3 weeks ago

I can reproduce it on my Debian system

<NOT PRESENT>
$ clang++-19 --print-file-name libc++.modules.json
/usr/lib/llvm-19/bin/../lib/libc++.modules.json

If you need the output of other commands please let me know.

ChuanqiXu9 commented 3 weeks ago

Thanks. But it might be hard for me to develop without being able to reproduce it...