ABataev marked an inline comment as done. ABataev added inline comments.
================ Comment at: clang/test/OpenMP/task_affinity_codegen.cpp:54 + // kmp_task_affinity_info_t affs[<num_elem>]; + // CHECK: [[AFFS_ADDR:%.+]] = alloca %struct.kmp_task_affinity_info_t, i64 [[NUM_ELEMS]], + // store i64 %21, i64* %__vla_expr0, align 8 ---------------- jdoerfert wrote: > I'm not so sure about dynamic allocas (without stack save/restore). If this > happens in a loop we can run into problems fast, right? There are stack saves and restores for this alloca, see the calls of `@llvm.stacksave()` and `@llvm.stackrestore()` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80240/new/ https://reviews.llvm.org/D80240 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
