[PATCH] D134556: [Clang][OpenMP] Fix run time crash when use_device_addr is used.

2022-09-27 Thread Jennifer Yu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. jyu2 marked an inline comment as done. Closed by commit rG30cc712eb6f2: [Clang][OpenMP] Fix run time crash when use_device_addr is used. (authored by jyu2). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D134556: [Clang][OpenMP] Fix run time crash when use_device_addr is used.

2022-09-27 Thread Alexey Bataev via Phabricator via cfe-commits
ABataev accepted this revision. ABataev added a comment. This revision is now accepted and ready to land. LG Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134556/new/ https://reviews.llvm.org/D134556 ___

[PATCH] D134556: [Clang][OpenMP] Fix run time crash when use_device_addr is used.

2022-09-27 Thread Jennifer Yu via Phabricator via cfe-commits
jyu2 updated this revision to Diff 463257. jyu2 added a comment. Merge duplicate code as Alexey suggested. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134556/new/ https://reviews.llvm.org/D134556 Files: clang/lib/CodeGen/CGOpenMPRuntime.cpp

[PATCH] D134556: [Clang][OpenMP] Fix run time crash when use_device_addr is used.

2022-09-26 Thread Jennifer Yu via Phabricator via cfe-commits
jyu2 added inline comments. Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:8561-8617 for (const auto *Cl : Clauses) { const auto *C = dyn_cast(Cl); if (!C) continue; for (const auto L : C->component_lists()) { OMPClauseMappableExprCo

[PATCH] D134556: [Clang][OpenMP] Fix run time crash when use_device_addr is used.

2022-09-23 Thread Alexey Bataev via Phabricator via cfe-commits
ABataev added inline comments. Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:8561-8617 for (const auto *Cl : Clauses) { const auto *C = dyn_cast(Cl); if (!C) continue; for (const auto L : C->component_lists()) { OMPClauseMappableExp

[PATCH] D134556: [Clang][OpenMP] Fix run time crash when use_device_addr is used.

2022-09-23 Thread Jennifer Yu via Phabricator via cfe-commits
jyu2 created this revision. jyu2 added reviewers: ABataev, jdoerfert, mikerice. jyu2 added a project: OpenMP. Herald added subscribers: guansong, yaxunl. Herald added a project: All. jyu2 requested review of this revision. Herald added subscribers: openmp-commits, cfe-commits, sstefan1. Herald adde