[Openmp-commits] [PATCH] D137828: RFC: [openmp] Don't assume a specific layout for alloca() in Windows arm64 __kmp_invoke_microtask

[Martin Storsjö via Phabricator via Openmp-commits]

mstorsjo abandoned this revision.
mstorsjo added a comment.

Abandoning this change for now.

Currently, microtasks with more than 6 arguments are broken when libomp is built with Clang for windows/arm64, while they work correctly (for any arbitrary number of arguments) when built with MSVC for the same configuration.

This patch improved things for Clang (for up to 15 arguments), while it also would have regressed the MSVC configuration down to only working for up to 15 arguments.

For Clang, D137827 <https://reviews.llvm.org/D137827> fixes the issue for any arbitrary number of arguments, by providing the same function implemented in gnu assembly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137828