[PATCH] D86155: [OpenMPOpt][SplitMemTransfer] Moving the "wait" down

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 18 16:21:05 PDT 2020


jdoerfert added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:719
+  Instruction *CurrentI = RuntimeCall;
+  bool IsWorthIt = false;
+  while ((CurrentI = CurrentI->getNextNode())) {
----------------
hamax97 wrote:
> jdoerfert wrote:
> > Can't you just check if CurrentI is RuntimeCall?
> I think it's never RuntimeCall. It starts with the instruction after RuntimeCall.
OK, keep it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86155



More information about the llvm-commits mailing list