[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