Skip to content

Commit 65a4674

Browse files
committed
Revert "[interop][SwiftToCxx] add -enable-experimental-cxx-interop-in-clang-header flag that lets you enable reverse interop only"
This reverts commit c193a63.
1 parent 8bce678 commit 65a4674

File tree

5 files changed

+0
-15
lines changed

5 files changed

+0
-15
lines changed

include/swift/Frontend/FrontendOptions.h

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -413,10 +413,6 @@ class FrontendOptions {
413413
/// header.
414414
llvm::Optional<ClangHeaderExposeBehavior> ClangHeaderExposedDecls;
415415

416-
/// Emit C++ bindings for the exposed Swift declarations in the generated
417-
/// clang header.
418-
bool EnableExperimentalCxxInteropInClangHeader = false;
419-
420416
/// \return true if the given action only parses without doing other compilation steps.
421417
static bool shouldActionOnlyParse(ActionType);
422418

include/swift/Option/Options.td

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -662,11 +662,6 @@ def experimental_c_foreign_reference_types :
662662
Flags<[FrontendOption, HelpHidden, ModuleInterfaceOption]>,
663663
HelpText<"Enable experimental C foreign references types (with reference coutning).">;
664664

665-
def enable_experimental_cxx_interop_in_clang_header :
666-
Flag<["-"], "enable-experimental-cxx-interop-in-clang-header">,
667-
Flags<[FrontendOption, NoDriverOption, HelpHidden]>,
668-
HelpText<"Enable experimental Swift to C++ interop code generation in generated Clang header">;
669-
670665
def experimental_cxx_stdlib :
671666
Separate<["-"], "experimental-cxx-stdlib">,
672667
Flags<[HelpHidden]>,

lib/Frontend/ArgsToFrontendOptionsConverter.cpp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -316,8 +316,6 @@ bool ArgsToFrontendOptionsConverter::convert(
316316
.Case("has-expose-attr", FrontendOptions::ClangHeaderExposeBehavior::HasExposeAttr)
317317
.Default(llvm::None);
318318
}
319-
Opts.EnableExperimentalCxxInteropInClangHeader =
320-
Args.hasArg(OPT_enable_experimental_cxx_interop_in_clang_header);
321319

322320
Opts.StrictImplicitModuleContext = Args.hasArg(OPT_strict_implicit_module_context,
323321
OPT_no_strict_implicit_module_context,

lib/PrintAsClang/PrintAsClang.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -556,7 +556,6 @@ bool swift::printAsClangHeader(raw_ostream &os, ModuleDecl *M,
556556
emitCxxConditional(os, [&] {
557557
// FIXME: Expose Swift with @expose by default.
558558
bool enableCxx = frontendOpts.ClangHeaderExposedDecls.has_value() ||
559-
frontendOpts.EnableExperimentalCxxInteropInClangHeader ||
560559
M->DeclContext::getASTContext().LangOpts.EnableCXXInterop;
561560
if (!enableCxx)
562561
return;

test/Interop/SwiftToCxx/expose-attr/expose-swift-decls-to-cxx.swift

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,6 @@
44

55
// RUN: %check-interop-cxx-header-in-clang(%t/expose.h -Wno-error=unused-function)
66

7-
// RUN: %target-swift-frontend %s -typecheck -module-name Expose -enable-experimental-cxx-interop-in-clang-header -clang-header-expose-decls=has-expose-attr -emit-clang-header-path %t/expose.h
8-
// RUN: %FileCheck %s < %t/expose.h
9-
107
// RUN: %empty-directory(%t)
118
// RUN: %target-swift-frontend %s -emit-module -module-name Expose -o %t
129
// RUN: %target-swift-frontend -parse-as-library %t/Expose.swiftmodule -typecheck -module-name Expose -enable-experimental-cxx-interop -clang-header-expose-decls=has-expose-attr -emit-clang-header-path %t/expose.h

0 commit comments

Comments
 (0)