[PATCH] D9386: llvm.noalias - CaptureTracking needs to look through them

[Jeroen Dobbelaere via llvm-commits]

jeroen.dobbelaere added a comment.

Maybe 'passthrough' should return a boolean, indicating that we need to stop or not. (see code comment)
The places where it is called could become something like: 'if (passthrough()) return;



================
Comment at: lib/Analysis/CaptureTracking.cpp:244
+        if (Count++ >= Threshold)
+          return Tracker->tooManyUses();
+
----------------
Moving this piece of code with a 'return' into a lambda results in a behavior change: instead of directly breaking the outer while loop when the Threshold is reached, it will now keep doing work with whatever is on the worklist. I don't think this change in behavior was intended.


https://reviews.llvm.org/D9386