[Openmp-commits] [PATCH] D158544: [OpenMP] Optimized trivial multiple edges from task dependency graph

[Romain PEREIRA via Phabricator via Openmp-commits]

rpereira-dev added inline comments.


================
Comment at: openmp/runtime/src/kmp_taskdeps.cpp:333
+                                                     kmp_depnode_t *sink,
+                                                     kmp_depnode_t *source) {
+  if (!source)
----------------
jdoerfert wrote:
> why is the order here swapped?
Order remains the same for callers; the patch only swaps in this prototype as i believe it was mistakenly inverted
(used to be `sink`  -> `source`, is now `source` -> `sink` )


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158544