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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
