[PATCH] D138630: [modules] Fix marking `ObjCMethodDecl::isOverriding` when there a no overrides.

Volodymyr Sapsai via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 24 11:16:56 PST 2022


vsapsai added a comment.

Thanks for the quick review! During the commit I will also fix the typo s/there a no overrides/there are no overrides/



================
Comment at: clang/test/Modules/override.m:30-34
+ at interface CheckOverrideImplementationOfCategory: NSObject
+ at end
+ at interface CheckOverrideImplementationOfCategory(CategoryImpl)
+- (void)potentialOverrideImplementationOfCategory;
+ at end
----------------
Actually, only this interface triggers the assertion. Added the rest for completeness and for better coverage.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138630



More information about the cfe-commits mailing list