[Openmp-commits] [PATCH] D82718: [OpenMP] Use primary context in CUDA plugin

Mon Jun 29 09:42:47 PDT 2020

ye-luo added a comment.

In D82718#2120164 <https://reviews.llvm.org/D82718#2120164>, @jdoerfert wrote:

> In D82718#2119143 <https://reviews.llvm.org/D82718#2119143>, @ye-luo wrote:
>
> > I'm not aware of any side effect but others may have different opinions. Since it is low effort to keep both, I made the choice to keep the old behavior accessible.
>
>
> More options mean more configurations we should test. Not that we do it right now but we should. Let's see if anyone else has an opinion about this.

If no one objects it, I'm happy to delete the old behavior.

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

https://reviews.llvm.org/D82718