[PATCH] D106720: [WIP][Attributor] AAFunctionReachability, Instruction reachability.

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a comment.

Add doxygen docu to the functions to describe what they do.



================
Comment at: llvm/lib/Transforms/IPO/AttributorAttributes.cpp:8691
+      return true;
+    };
+
----------------
Don't accumulate pointers to containers but content instead.


================
Comment at: llvm/lib/Transforms/IPO/AttributorAttributes.cpp:8835-8836
+    for (auto QueryPair : InstQueries) {
+      Change = Change | checkReachableBackwards(A, QueryPair.second);
+    }
+
----------------
`|=`, also elsewhere
no braces.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106720