[all-commits] [llvm/llvm-project] 7ad693: [modules] Update visibility for merged ObjCProtoco...

[Volodymyr Sapsai via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7ad693a322c1b6765f5d06559c2bd73cc3938aaf
      https://github.com/llvm/llvm-project/commit/7ad693a322c1b6765f5d06559c2bd73cc3938aaf
  Author: Volodymyr Sapsai <vsapsai at apple.com>
  Date:   2021-11-08 (Mon, 08 Nov 2021)

  Changed paths:
    M clang/lib/Serialization/ASTReaderDecl.cpp
    A clang/test/Modules/merge-objc-protocol-visibility.m

  Log Message:
  -----------
  [modules] Update visibility for merged ObjCProtocolDecl definitions.

Merge definition visibility the same way we do for other decls. Without
the fix the added test emits `-Wobjc-method-access` as it cannot find a
visible protocol. Make this warning `-Werror` so the test would fail
when protocol visibility regresses.

rdar://83600696

Differential Revision: https://reviews.llvm.org/D111860