[PATCH] D126189: [C++20][Modules] Build module static initializers per P1874R1.

Iain Sandoe via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 11 09:42:53 PDT 2022


iains added a comment.

In D126189#3642777 <https://reviews.llvm.org/D126189#3642777>, @iains wrote:

> In D126189#3642762 <https://reviews.llvm.org/D126189#3642762>, @JDevlieghere wrote:
>
>> This breaks TestDataFormatterLibcxxSpan.py on GreenDragon:
>>
>>   Undefined symbols for architecture x86_64:

<snip>..

>>   "__ZGIW6vector", referenced from:
>>       __GLOBAL__sub_I_main.cpp in main.o
>>
>>   https://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/45207/
>
> ack, (I develop on macOS so usually catch these things)...
> do you need me to revert the commit - or can we try to find out what's broken and fix it?

hmm there seems to be a compiler error, which looks somewhat unrelated to the active patch:

  /Users/buildslave/jenkins/workspace/lldb-cmake/llvm-project/cross-project-tests/debuginfo-tests/clang_llvm_roundtrip/simplified_template_names.cpp:125:27: error: no type named 'size_t' in namespace 'std'
    void* operator new(std::size_t, T) {
                       ~~~~~^
  /Users/buildslave/jenkins/workspace/lldb-cmake/llvm-project/cross-project-tests/debuginfo-tests/clang_llvm_roundtrip/simplified_template_names.cpp:132:29: error: no type named 'size_t' in namespace 'std'
    void* operator new[](std::size_t, T) {


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126189



More information about the cfe-commits mailing list