[PATCH] D118599: [C++20][Modules][8/8] Amend module visibility rules for partitions.

Iain Sandoe via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Feb 27 23:43:20 PST 2022


iains marked an inline comment as done.
iains added inline comments.


================
Comment at: clang/test/Modules/cxx20-10-1-ex2.cpp:15-17
-// Not expected to work yet.
 //  %clang_cc1 -std=c++20 -emit-module-interface %t/std10-1-ex2-tu4.cpp \
 //   -fmodule-file=%t/B.pcm  -o %t/B_X2.pcm
----------------
h-vetinari wrote:
> Should these two tests now be switched on? Missing a `RUN: ` for that IIUC?
good catch, thanks!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D118599/new/

https://reviews.llvm.org/D118599



More information about the cfe-commits mailing list