[PATCH] D80240: [OPENMP50]Initial codegen for 'affinity' clauses.

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 29 13:41:14 PDT 2020


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





More information about the cfe-commits mailing list