mordante wrote:

> LGTM with a few nits. Can you double check modules.json is inside `lib/` 
> (`usr/lib/x86_64-linux-gnu{{/|\\}}modules.json`) instead of some directory 
> under `include`?

Thanks for the review. The json will indeed be installed in the lib dir in 
libc++.

https://github.com/llvm/llvm-project/pull/76451
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to