[PATCH] D87260: [OpenMPOpt][SplitMemTransfer] Can move setup instructions upwards?

Tue Sep 8 08:08:09 PDT 2020

jdoerfert added a comment.

In D87260#2261006 <https://reviews.llvm.org/D87260#2261006>, @hamax97 wrote:

> In D87260#2260098 <https://reviews.llvm.org/D87260#2260098>, @jdoerfert wrote:
>
>> Tests?
>
> The regression test for this is `hide_mem_transfer_latency.ll`.  But there's still the issue with the `AAManager`, so the test has the fixmes of where the set up instructions should be moved.

But it seems not to include any changes to the test. why not? because this doesn't have an impact without resolving the issue you mentioned?

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

https://reviews.llvm.org/D87260