[PATCH] D129748: [C++20] [Modules] Warn for the use of preferred_name in modules

Thu Jul 21 06:03:05 PDT 2022

erichkeane added a comment.

In D129748#3667932 <https://reviews.llvm.org/D129748#3667932>, @ChuanqiXu wrote:

> @erichkeane @aaron.ballman @tahonermann I get another idea. We could tell the user honestly that we couldn't handle it (temporarily). We could still modularize the STL by a trick: https://github.com/ChuanqiXu9/stdmodules/blob/e81f4e9e74f96021f2e45c48f44da93e806c4524/Makefile#L3
>
> In this way, we don't need to worry about the dirty implementation pollutes the sources. (I think it is not bad to emit a warning to tell the user we couldn't do something (now).)

Thats a heck of an "STL Trick".  The diagnostic itself needs some rewording, and I don't know if we do diagnostics for our implementation deficiencies.

I still consider the 'disable the attribute' as perhaps the 'best idea' for now other than fixing the bug, but had suggested earlier doing it NOT at the SemaDeclAttr level, but to do it at the ASTWriter-when-in-modules level.

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D129748/new/

https://reviews.llvm.org/D129748