[clang] [Clang] Implement P3034R1 Module Declarations Shouldn’t be Macros (PR #90574)

[via cfe-commits]

yronglin wrote:

> This commit seems to have broken well-formed code in thrift:
> 
> ```
> fbcode/thrift/compiler/generate/t_py_generator.cc:881:20: error: declaration of variable 'module' with deduced type 'const auto &' requires an initializer
>   881 |   for (const auto &module : modules) {
>       |                    ^
> fbcode/thrift/compiler/generate/t_py_generator.cc:881:27: error: expected ';' in 'for' statement specifier
>   881 |   for (const auto &module : modules) {
>       |                           ^
> fbcode/thrift/compiler/generate/t_py_generator.cc:881:27: error: expected expression
> fbcode/thrift/compiler/generate/t_py_generator.cc:881:27: error: expected ';' in 'for' statement specifier
> 4 errors generated.
> ```

This is clearer, I think I may have figured out where the problem is.

https://github.com/llvm/llvm-project/pull/90574