[all-commits] [llvm/llvm-project] db987b: [Modules] Remove unnecessary check when generating...

[Chuanqi Xu via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: db987b9589be1eb604fcb74c85b410469e31485f
      https://github.com/llvm/llvm-project/commit/db987b9589be1eb604fcb74c85b410469e31485f
  Author: Chuanqi Xu <yedeng.yd at linux.alibaba.com>
  Date:   2023-03-09 (Thu, 09 Mar 2023)

  Changed paths:
    M clang/include/clang/Serialization/ASTWriter.h
    M clang/lib/Serialization/ASTWriter.cpp
    A clang/test/Modules/pr61065.cppm

  Log Message:
  -----------
  [Modules] Remove unnecessary check when generating name lookup table in ASTWriter

Close https://github.com/llvm/llvm-project/issues/61065.

We will avoid writing the names from external AST naturally. But
currently its check is often false positive since we may have already
marked the declarations as external but
DeclContext::hasNeedToReconcileExternalVisibleStorage would be false
after reconciling.

Tested with libcxx's modular build.

This patch can improve 8% compilation time in an internal workloads.