[all-commits] [llvm/llvm-project] b7526c: [NFC][clang][IFS] Adding braces to if-statement as...

Puyan Lotfi via All-commits all-commits at lists.llvm.org
Mon Jan 13 13:04:50 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: b7526cc21ce55c8b53250df3d659fbdae3f894a7
      https://github.com/llvm/llvm-project/commit/b7526cc21ce55c8b53250df3d659fbdae3f894a7
  Author: Puyan Lotfi <puyan at puyan.org>
  Date:   2020-01-13 (Mon, 13 Jan 2020)

  Changed paths:
    M clang/lib/Frontend/InterfaceStubFunctionsConsumer.cpp

  Log Message:
  -----------
  [NFC][clang][IFS] Adding braces to if-statement as prep for D71301.

Just trying to make https://reviews.llvm.org/D71301 look cleaner.


  Commit: bd8c8827d96f09be502f0da6897c1aef89e45c30
      https://github.com/llvm/llvm-project/commit/bd8c8827d96f09be502f0da6897c1aef89e45c30
  Author: Puyan Lotfi <puyan at puyan.org>
  Date:   2020-01-13 (Mon, 13 Jan 2020)

  Changed paths:
    M clang/lib/Frontend/InterfaceStubFunctionsConsumer.cpp
    A clang/test/InterfaceStubs/blocks.c
    A clang/test/InterfaceStubs/lambda.cpp

  Log Message:
  -----------
  [clang][IFS] Prevent Clang-IFS from Leaking symbols from inside a block.

Built libdispatch with clang interface stubs. Ran into some block
related issues. Basically VarDecl symbols can leak out because I wasn't
checking the case where a VarDecl is contained inside a BlockDecl
(versus a method or function).

This patch checks that a VarDecl is not a child decl of a BlockDecl.
This patch also does something very similar for c++ lambdas as well.

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


Compare: https://github.com/llvm/llvm-project/compare/c1b13a1b1771...bd8c8827d96f


More information about the All-commits mailing list