[all-commits] [llvm/llvm-project] 3cca52: [C++20] [Modules] Warn for duplicated decls in mut...

Chuanqi Xu via All-commits all-commits at lists.llvm.org
Fri Aug 23 02:43:08 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 3cca522d21876da36145655bc14f334035b4265d
      https://github.com/llvm/llvm-project/commit/3cca522d21876da36145655bc14f334035b4265d
  Author: Chuanqi Xu <yedeng.yd at linux.alibaba.com>
  Date:   2024-08-23 (Fri, 23 Aug 2024)

  Changed paths:
    M clang/include/clang/Basic/DiagnosticSerializationKinds.td
    M clang/include/clang/Serialization/ASTReader.h
    M clang/lib/Serialization/ASTReader.cpp
    M clang/lib/Serialization/ASTReaderDecl.cpp
    A clang/test/Modules/warn-duplicated-decls-in-module-units.cppm

  Log Message:
  -----------
  [C++20] [Modules] Warn for duplicated decls in mutliple module units (#105799)

It is a long standing issue that the duplicated declarations in multiple
module units would cause the compilation performance to get slowed down.
And there are many questions or issue reports. So I think it is better
to add a warning for it.

And given this is not because the users' code violates the language
specification or any best practices, the warning is disabled by default
even if `-Wall` is specified. The users need to specify the warning
explcitly or use `Weverything`.

The documentation will add separately.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list