This revision was automatically updated to reflect the committed changes.
Closed by commit rGbad44d5f39bc: [OpenMP] Add RTL function for getting number
of threads in block. (authored by jhuber6).
Changed prior to commit:
https://reviews.llvm.org/D111475?vs=378393&id=378397#toc
Repository:
rG
jhuber6 updated this revision to Diff 378393.
jhuber6 added a comment.
Fixing tests.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D111475/new/
https://reviews.llvm.org/D111475
Files:
clang/lib/CodeGen/CGOpenMPRuntimeAMDGCN.cpp
clang/lib/CodeGe
JonChesterfield accepted this revision.
JonChesterfield added a comment.
This revision is now accepted and ready to land.
Great! I like the decrease in complexity of the compiler. Pretty close to
dropping the nvptx and amdgpu subclasses entirely
Repository:
rG LLVM Github Monorepo
CHANGES SI
jhuber6 created this revision.
jhuber6 added a reviewer: jdoerfert.
Herald added subscribers: guansong, tpr, yaxunl, jvesely, jholewinski.
jhuber6 requested review of this revision.
Herald added subscribers: openmp-commits, cfe-commits, sstefan1.
Herald added projects: clang, OpenMP.
This patch ad