[PATCH] D83316: [OpenMPOpt][WIP] Structure for unittests

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 18 21:44:45 PDT 2020


jdoerfert added a comment.

In D83316#2160742 <https://reviews.llvm.org/D83316#2160742>, @hamax97 wrote:

> In D83316#2160625 <https://reviews.llvm.org/D83316#2160625>, @jdoerfert wrote:
>
> > Wow, this required more machinery than I expected.. thanks!
> >
> > Should we merge this with the `MemoryTransfer` logic so we can test it?
>
>
> I already tested it. When we agree on D82719 <https://reviews.llvm.org/D82719>, we should commit that first and then this I guess. Or do you mean adding this patch to that one before commiting?


I mean merging them into one and commiting them together.


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

https://reviews.llvm.org/D83316





More information about the llvm-commits mailing list