On 2/7/25 6:15 AM, Nathaniel Shead wrote:
Tested on x86_64-pc-linux-gnu, OK for trunk?
OK.
-- >8 -- This testcase wasn't running, because adl-5_a had the wrong extension. adl-5_d should have been reporting an error because 'frob' is only visible from within the 'hidden' module but this was missed. gcc/testsuite/ChangeLog: * g++.dg/modules/adl-5_a.c: Move to... * g++.dg/modules/adl-5_a.C: ...here. * g++.dg/modules/adl-5_d.C: Add errors. Signed-off-by: Nathaniel Shead <[email protected]> --- gcc/testsuite/g++.dg/modules/{adl-5_a.c => adl-5_a.C} | 0 gcc/testsuite/g++.dg/modules/adl-5_d.C | 5 +++-- 2 files changed, 3 insertions(+), 2 deletions(-) rename gcc/testsuite/g++.dg/modules/{adl-5_a.c => adl-5_a.C} (100%) diff --git a/gcc/testsuite/g++.dg/modules/adl-5_a.c b/gcc/testsuite/g++.dg/modules/adl-5_a.C similarity index 100% rename from gcc/testsuite/g++.dg/modules/adl-5_a.c rename to gcc/testsuite/g++.dg/modules/adl-5_a.C diff --git a/gcc/testsuite/g++.dg/modules/adl-5_d.C b/gcc/testsuite/g++.dg/modules/adl-5_d.C index 9c75b6d14a7..09760c5ad01 100644 --- a/gcc/testsuite/g++.dg/modules/adl-5_d.C +++ b/gcc/testsuite/g++.dg/modules/adl-5_d.C @@ -7,10 +7,11 @@ int main () { X x (2);- if (frob (x) != 2)+ if (frob (x) != 2) // { dg-error "not declared in" } return 1;- if (TPL (x) != 2)+ // { dg-regexp "\n\[^\n]*adl-5_a.C:9:15: error: 'frob' was not declared in this scope$" } + if (TPL (x) != 2) // { dg-message "required from here" } return 2;return 0;
