[PATCH] D116595: [clang][sema] Add missing diagnostic parameter

Timm Bäder via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 14 01:08:04 PST 2022


tbaeder added inline comments.


================
Comment at: clang/test/Modules/cxx20-export-import.cpp:1-2
+
+// RUN: %clang_cc1 -std=c++20 -fmodules -fmodules-cache-path=%t -fimplicit-module-maps -I%S/Inputs -stdlib=libc++ -verify %s
+export import dummy; // expected-error {{export declaration can only be used within a module interface unit after the module declaration}}
----------------
aaron.ballman wrote:
> Also, does this test require `-fmodules-cache-path=%t` or `-fimplicit-module-maps`?
They are all needed (including the `-I`). I could only remove the `-stdlib=libc++`, removing any of the others makes the test fail :(


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

https://reviews.llvm.org/D116595



More information about the cfe-commits mailing list