[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