[clang] [Cygwin] Reduced number of inline elements of CallArgList. (PR #74977)

徐持恒 Xu Chiheng via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 12 09:14:52 PST 2023


xu-chiheng wrote:

> > I have build scripts and patches at: https://github.com/xu-chiheng/Note
> > […](#)
> > On Mon, Dec 11, 2023 at 11:25 PM Yingchi Long _**@**_._**> wrote: When bootstrapping Clang on Cygwin, there are various weird problems. What are those "weired problems" Reducing the size of CallArgList solve them all. And why reduce the size of CallArgList solve them all? Could you kindly elaborate? — Reply to this email directly, view it on GitHub <[#74977 (comment)](https://github.com/llvm/llvm-project/pull/74977#issuecomment-1850302699)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA5RCUGAE5BNGG5HBCLW66DYI4QXJAVCNFSM6AAAAABAOLH3Y6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJQGMYDENRZHE . You are receiving this because you authored the thread.Message ID: _**@**_.**_>
> 
> I see, can you just paste logs(e.g. compiler warnings) here or provide a small reproducer? (e.g step-by-step instructions to reproduce the issue).
> 
> I think it will be nice if the patch really improves llvm portability, but it is hard to reproduce bugs via a set of undocumented scripts & unknown build environments.

No, currently. It was several months ago.  I don't remember the details. Basically speaking,  it seems that some fields of CallArgList or other types are overwritten, because CallArgList size exceed an unknown limit, leading to various weird errors and failed bootstrap.  This need to be invesgated, but I'm not very familiar with the code. 

This was discovered by bisect. Clang 16 can bootstrap without this patch.


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


More information about the cfe-commits mailing list