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

2020-06-02 Thread Alexey Bataev via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG89d9dba2c688: [OPENMP50]Initial codegen for 'affinity' clauses. (authored by ABataev). Changed prior to commit: https://reviews.llvm.org/D80240?vs=267254&id=267895#toc Repository: rG LLVM Github Mono

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

2020-05-29 Thread Johannes Doerfert via Phabricator via cfe-commits
jdoerfert accepted this revision. jdoerfert added subscribers: jhuber6, clementval, fghanim, DavidTruby. jdoerfert added a comment. This revision is now accepted and ready to land. LGTM. Let's wait until D80222 landed :) Comment at: clang/lib/C

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

2020-05-29 Thread Alexey Bataev via Phabricator via cfe-commits
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[]; + // CHECK: [[AFFS_ADDR:%.+]] = alloca %struct.kmp_task_affinity_info_t, i64 [[NUM_ELEMS]], + // store i64

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

2020-05-29 Thread Johannes Doerfert via Phabricator via cfe-commits
jdoerfert added a comment. Few comments, the alloca question is my only real concern. Parts of this are trivial NFC and can go in to make the patch smaller. Parts won't be needed after D80222 landed :) Comment at: clang/lib/CodeGen/CGOpenMPRu

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

2020-05-29 Thread Alexey Bataev via Phabricator via cfe-commits
ABataev updated this revision to Diff 267254. ABataev added a comment. Rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80240/new/ https://reviews.llvm.org/D80240 Files: clang/lib/CodeGen/CGOpenMPRuntime.cpp clang/lib/CodeGen/CGOpenMPRunti

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

2020-05-19 Thread Alexey Bataev via Phabricator via cfe-commits
ABataev created this revision. ABataev added a reviewer: jdoerfert. Herald added subscribers: llvm-commits, sstefan1, guansong, yaxunl. Herald added projects: clang, LLVM. Added initial codegen for 'affinity' clauses on task directives. Emits next code: kmp_task_affinity_info_t affs[]; voi