[all-commits] [llvm/llvm-project] 442a80: [clang] Don't make synthesized accessor stub funct...

Raphael Isemann via All-commits all-commits at lists.llvm.org
Tue Aug 11 07:24:26 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 442a80292d50d895396eb14418bd471e7da68fd0
      https://github.com/llvm/llvm-project/commit/442a80292d50d895396eb14418bd471e7da68fd0
  Author: Raphael Isemann <teemperor at gmail.com>
  Date:   2020-08-11 (Tue, 11 Aug 2020)

  Changed paths:
    M clang/lib/Sema/SemaDeclObjC.cpp

  Log Message:
  -----------
  [clang] Don't make synthesized accessor stub functions visible twice

`addDecl` is making the Decl visible, so there is no need to make it explicitly
visible again. Making it visible twice will also make the lookup storage less
efficient and potentially lead to crashes, see D84827 for that.

Reviewed By: aprantl

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




More information about the All-commits mailing list