From 26bbb8700bb0ea0ce29e4158e5aa7999ab0d5386 Mon Sep 17 00:00:00 2001 From: Jan Svoboda Date: Wed, 14 Apr 2021 14:35:03 +0200 Subject: [clang] Implement CompilerInvocation copy assignment This patch implements the copy assignment for `CompilerInvocation`. Eventually, the deep-copy operation will be moved into a `clone()` method (D100460), but until then, this is necessary for basic ergonomics. Depends on D100455. Reviewed By: Bigcheese Differential Revision: https://reviews.llvm.org/D100473 --- clang/unittests/Frontend/CompilerInvocationTest.cpp | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) (limited to 'clang/unittests/Frontend/CompilerInvocationTest.cpp') diff --git a/clang/unittests/Frontend/CompilerInvocationTest.cpp b/clang/unittests/Frontend/CompilerInvocationTest.cpp index 5062a2e..0baf17f 100644 --- a/clang/unittests/Frontend/CompilerInvocationTest.cpp +++ b/clang/unittests/Frontend/CompilerInvocationTest.cpp @@ -97,7 +97,7 @@ TEST(ContainsN, Two) { ASSERT_THAT(Array, ContainsN(StrEq("x"), 2)); } -// Copy constructor performs a deep copy of reference-counted pointers. +// Copy constructor/assignment perform deep copy of reference-counted pointers. TEST(CompilerInvocationTest, DeepCopyConstructor) { CompilerInvocation A; @@ -109,6 +109,17 @@ TEST(CompilerInvocationTest, DeepCopyConstructor) { ASSERT_EQ(A.getAnalyzerOpts()->Config["Key"], "Old"); } +TEST(CompilerInvocationTest, DeepCopyAssignment) { + CompilerInvocation A; + A.getAnalyzerOpts()->Config["Key"] = "Old"; + + CompilerInvocation B; + B = A; + B.getAnalyzerOpts()->Config["Key"] = "New"; + + ASSERT_EQ(A.getAnalyzerOpts()->Config["Key"], "Old"); +} + // Boolean option with a keypath that defaults to true. // The only flag with a negative spelling can set the keypath to false. -- cgit v1.1