xujuntwt95329 added a comment.

In D112439#3102548 <https://reviews.llvm.org/D112439#3102548>, @teemperor wrote:

> In D112439#3102533 <https://reviews.llvm.org/D112439#3102533>, @xujuntwt95329 
> wrote:
>
>> In D112439#3102506 <https://reviews.llvm.org/D112439#3102506>, @teemperor 
>> wrote:
>>
>>> In D112439#3098307 <https://reviews.llvm.org/D112439#3098307>, 
>>> @xujuntwt95329 wrote:
>>>
>>>> Seems that patch can't build by CI because it is based on this patch. In 
>>>> my understanding we need to merge this patch firstly and rebase that NFC 
>>>> patch to let CI work, right?
>>>
>>> You can set parent/child revisions in Phabricator that should handle this 
>>> situation (not sure if the premerge-checks are respecting that, but that's 
>>> how Phab usually manages unmerged dependencies).
>>>
>>> But to be clear, the premerge checks on Phabricator are *not* building or 
>>> testing LLDB at all. See the CI log:
>>>
>>>   INFO    projects: {'lldb'}
>>>   INFO    all excluded projects(*) {'check-cxxabi', 'flang', 'libunwind', 
>>> 'compiler-rt', 'cross-project-tests', 'libcxx', 'libcxxabi', 'libc', 
>>> 'openmp', 'lldb'}
>>>   INFO    effective projects list set()
>>
>> Thanks a lot for the detailed information!
>>
>> BTW, I see there are some test failure reported by CI, but I can't reproduce 
>> them in my local environment (X86_64 Ubuntu and Windows), is there any 
>> docker images to simulate the CI environment?
>
> Can you try a Release + Assert build? I'm looking at the failures at the 
> moment too and it seems to only fail in non-Debug builds.

Sure, I'll try the `release + assert` build. Thanks again!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112439

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to