[all-commits] [llvm/llvm-project] c9537b: [lldb/debugserver] Include TargetConditionals.h wh...

fredriss via All-commits all-commits at lists.llvm.org
Sat May 9 10:35:26 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: c9537b9cc862b7ec1ba502043e42bc6ebd24dec0
      https://github.com/llvm/llvm-project/commit/c9537b9cc862b7ec1ba502043e42bc6ebd24dec0
  Author: Fred Riss <friss at apple.com>
  Date:   2020-05-09 (Sat, 09 May 2020)

  Changed paths:
    M lldb/tools/debugserver/source/MacOSX/MachProcess.mm

  Log Message:
  -----------
  [lldb/debugserver] Include TargetConditionals.h where needed

MachProcess.mm uses a TARGET_OS_ macro without directly including
TargetConditionals.h. This currently works as we get the header
as an indirect dependency, but might not in the future.

I just spent some time investigating an internal regression
caused by a similar issue, so I audited the codebase for such
cases.




More information about the All-commits mailing list