Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[APINotes] Remove unused API #98943

Merged
merged 1 commit into from
Jul 16, 2024
Merged

Conversation

egorzhdan
Copy link
Contributor

This method is not actually used anywhere.

This method is not actually used anywhere.
@egorzhdan egorzhdan requested a review from compnerd July 15, 2024 18:38
@llvmbot llvmbot added the clang Clang issues not falling into any other category label Jul 15, 2024
@llvmbot
Copy link
Member

llvmbot commented Jul 15, 2024

@llvm/pr-subscribers-clang

Author: Egor Zhdan (egorzhdan)

Changes

This method is not actually used anywhere.


Full diff: https://github.com/llvm/llvm-project/pull/98943.diff

1 Files Affected:

  • (modified) clang/include/clang/APINotes/Types.h (-7)
diff --git a/clang/include/clang/APINotes/Types.h b/clang/include/clang/APINotes/Types.h
index daf2f1897f46b..b389aa8d56f16 100644
--- a/clang/include/clang/APINotes/Types.h
+++ b/clang/include/clang/APINotes/Types.h
@@ -263,13 +263,6 @@ class ContextInfo : public CommonTypeInfo {
     SwiftObjCMembers = Value.value_or(false);
   }
 
-  /// Strip off any information within the class information structure that is
-  /// module-local, such as 'audited' flags.
-  void stripModuleLocalInfo() {
-    HasDefaultNullability = false;
-    DefaultNullability = 0;
-  }
-
   friend bool operator==(const ContextInfo &, const ContextInfo &);
 
   ContextInfo &operator|=(const ContextInfo &RHS) {

Copy link
Member

@compnerd compnerd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good. I wonder if this was used at one point and we removed that?

@egorzhdan
Copy link
Contributor Author

Yeah, I can't see any usages of this in the past few years. It was probably used a long while ago.

@egorzhdan egorzhdan merged commit 4348f32 into main Jul 16, 2024
9 checks passed
@egorzhdan egorzhdan deleted the users/egorzhdan/apinotes-remove-unused branch July 16, 2024 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang Clang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants