[llvm] LAA: refactor analyzeLoop to return bool (NFC) (PR #93824)

[Nikita Popov via llvm-commits]

================
@@ -2571,12 +2567,20 @@ void LoopAccessInfo::analyzeLoop(AAResults *AA, LoopInfo *LI,
     LLVM_DEBUG(
         dbgs() << "LAA: A loop annotated parallel, ignore memory dependency "
                << "checks.\n");
-    CanVecMem = true;
-    return;
+    return true;
   }
 
   for (LoadInst *LD : Loads) {
     Value *Ptr = LD->getPointerOperand();
+
+    // See if there is an unsafe dependency between a load to a uniform address
+    // and store to the same uniform address.
+    if (UniformStores.count(Ptr)) {
+      LLVM_DEBUG(dbgs() << "LAA: Found an unsafe dependency between a uniform "
+                           "load and uniform store to the same address!\n");
+      HasLoadStoreDependenceInvolvingLoopInvariantAddress = true;
+    }
----------------
nikic wrote:

Why did this code move?

https://github.com/llvm/llvm-project/pull/93824