[PATCH] D80917: [OpenMP][AMDGCN] Support OpenMP offloading for AMDGCN architecture - Part 2

Saiyedul Islam via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 10 11:05:16 PDT 2020


saiislam added a comment.

In D80917#2085377 <https://reviews.llvm.org/D80917#2085377>, @jdoerfert wrote:

> I am fine with this. @arsenm feel free to accept.
>
>  ---
>
> In D80917#2085138 <https://reviews.llvm.org/D80917#2085138>, @saiislam wrote:
>
> > Ping.
>
>
> Afaik, pings should be reserved to ~1 week of inactivity. I know this can be frustrating, but we also need to consider noise for reviewers.


Ok. I will keep in mind next time. Thanks :-)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80917/new/

https://reviews.llvm.org/D80917





More information about the cfe-commits mailing list