tianshilei1992 added a comment.

In D142297#4082242 <https://reviews.llvm.org/D142297#4082242>, @mstorsjo wrote:

> This change broke the `parallel/bug54082.c` testcase in the OpenMP runtime 
> test set, when running on Windows (both mingw and MSVC configurations, and 
> happening both in i386 and x86_64 builds), see e.g. 
> https://github.com/mstorsjo/llvm-mingw/actions/runs/4011290068/jobs/6891673869.
>  It's failing with error code `0xC00000FD`, which means 
> `STATUS_STACK_OVERFLOW`. Is this change missing some aspect that differs 
> between the unix and Windows ABIs?
>
> So please don't backport this to the release branch right now. And I'd 
> appreciate if you would consider to revert it if there's no immediately 
> obvious fix in sight.

Thanks for the information. I'll install a Windows VM and try to fix it and get 
the back ported to the release branch as this patch has already been included.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142297

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

Reply via email to