[all-commits] [llvm/llvm-project] fdec9f: [Clang] Fix export declaration diagnostic message ...

[duhbbx via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: fdec9fd4f81fd336d6d5d50bbd48cd0e095f46b9
      https://github.com/llvm/llvm-project/commit/fdec9fd4f81fd336d6d5d50bbd48cd0e095f46b9
  Author: duhbbx <duhbbx at gmail.com>
  Date:   2025-07-16 (Wed, 16 Jul 2025)

  Changed paths:
    M clang/include/clang/Basic/DiagnosticSemaKinds.td
    M clang/test/CXX/drs/cwg8xx.cpp
    M clang/test/CXX/module/dcl.dcl/dcl.module/dcl.module.interface/p1.cppm
    M clang/test/CXX/module/module.interface/p1.cpp
    M clang/test/Modules/cxx20-10-2-ex1.cpp
    M clang/test/Modules/cxx20-export-import.cpp
    M clang/test/Modules/cxx20-import-diagnostics-a.cpp
    M clang/test/Modules/export-in-non-modules.cpp

  Log Message:
  -----------
  [Clang] Fix export declaration diagnostic message (#149059)

Change the error message from "export declaration can only be used
within a module purview" to "export declaration can only be used within
a module interface" to be technically accurate.

The previous message was misleading because export declarations are
actually within a module purview when used in module implementation
units, but they are only allowed in module interface units.

This addresses the issue pointed out in GitHub issue #149008 where
Bigcheese noted that the diagnostic wording was incorrect.

Fixes #149008



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