Sequel patch to https://reviews.llvm.org/D111293.
Remove call to CodeGenFunction::InitTempAlloca() from OpenMP related
codegen part.
Also remove the metadata !llvm.access.group from the updated lit
tests.
Paths
| Differential D111316
[CFE][Codegen][In-progress] Remove CodeGenFunction::InitTempAlloca() ClosedPublic Authored by hsmhsm on Oct 7 2021, 8:16 AM.
Details Summary Sequel patch to https://reviews.llvm.org/D111293. Remove call to CodeGenFunction::InitTempAlloca() from OpenMP related Also remove the metadata !llvm.access.group from the updated lit
Diff Detail
Unit TestsFailed
Event Timelinehsmhsm added a parent revision: D111293: [CFE][Codegen][In-progress] Remove CodeGenFunction::InitTempAlloca().Oct 7 2021, 8:18 AM hsmhsm added a child revision: D111324: [CFE][Codegen] Remove CodeGenFunction::InitTempAlloca().Oct 7 2021, 10:13 AM Comment Actions Move OpenMP GPU offloading codegen related changes from https://reviews.llvm.org/D111324 to here. Comment Actions This mostly looks good, but I'd like Alexey Bataev to sign off, since he authored the OpenMP support.
This revision is now accepted and ready to land.Oct 11 2021, 2:20 PM Closed by commit rGf7de6962c85b: [CFE][Codegen][In-progress] Remove CodeGenFunction::InitTempAlloca() (authored by hsmhsm). · Explain WhyOct 11 2021, 9:32 PM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 377858 clang/lib/CodeGen/CGOpenMPRuntime.cpp
clang/test/OpenMP/distribute_parallel_for_if_codegen.cpp
clang/test/OpenMP/distribute_parallel_for_simd_if_codegen.cpp
clang/test/OpenMP/parallel_if_codegen.cpp
clang/test/OpenMP/parallel_if_codegen_PR51349.cpp
clang/test/OpenMP/parallel_master_taskloop_codegen.cpp
clang/test/OpenMP/parallel_master_taskloop_simd_codegen.cpp
clang/test/OpenMP/target_codegen_global_capture.cpp
clang/test/OpenMP/target_parallel_for_simd_codegen.cpp
clang/test/OpenMP/target_parallel_if_codegen.cpp
clang/test/OpenMP/target_teams_distribute_parallel_for_if_codegen.cpp
clang/test/OpenMP/target_teams_distribute_parallel_for_simd_if_codegen.cpp
clang/test/OpenMP/task_codegen.c
clang/test/OpenMP/teams_distribute_parallel_for_if_codegen.cpp
clang/test/OpenMP/teams_distribute_parallel_for_simd_if_codegen.cpp
clang/test/OpenMP/vla_crash.c
|
This seems innocuous since ZeroAddrBound does not otherwise change in the function, assuming the runtime call doesn't modify it.