[PATCH] D81433: AMDGPU: Restrict the number of instructions to scan for getPointerDependencyFrom

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUAnnotateUniformValues.cpp:109
+  // Maximum number of instructions to search in a block (default 100).
+  auto Limit = MDR->getDefaultBlockScanLimit();
   for (auto &BB : Checklist) {
----------------
I don't see how this changes anything. This is what the default null case does?

  unsigned DefaultLimit = getDefaultBlockScanLimit();
  if (!Limit)
    Limit = &DefaultLimit;



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

https://reviews.llvm.org/D81433