ilya-biryukov wrote: Oh, one last thing! @usx95 maybe add a test for the visibility check? Various tests with modules (search for files with the `.cppm` extension in the tests) should give a good starting point. https://github.com/llvm/llvm-project/pull/68922