[PATCH] D140097: [MemDep] Reduce block limit

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 11 06:48:12 PST 2023


nikic updated this revision to Diff 488195.
nikic added a comment.

Default to 300 blocks.

Compile-time results for this are a bit underwhelming, but I guess it doesn't hurt to start conservatively: http://llvm-compile-time-tracker.com/compare.php?from=cc845e9de8c87c74d494f4e90e8fcf4fca264989&to=1b691111c3e80929268c8b749470d2b5bfad6f9d&stat=instructions:u

Did you happen to also try 200 blocks?


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

https://reviews.llvm.org/D140097

Files:
  llvm/lib/Analysis/MemoryDependenceAnalysis.cpp


Index: llvm/lib/Analysis/MemoryDependenceAnalysis.cpp
===================================================================
--- llvm/lib/Analysis/MemoryDependenceAnalysis.cpp
+++ llvm/lib/Analysis/MemoryDependenceAnalysis.cpp
@@ -76,9 +76,9 @@
              "dependency analysis (default = 100)"));
 
 static cl::opt<unsigned>
-    BlockNumberLimit("memdep-block-number-limit", cl::Hidden, cl::init(1000),
+    BlockNumberLimit("memdep-block-number-limit", cl::Hidden, cl::init(300),
                      cl::desc("The number of blocks to scan during memory "
-                              "dependency analysis (default = 1000)"));
+                              "dependency analysis (default = 300)"));
 
 // Limit on the number of memdep results to process.
 static const unsigned int NumResultsLimit = 100;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D140097.488195.patch
Type: text/x-patch
Size: 810 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230111/3abbe76a/attachment.bin>


More information about the llvm-commits mailing list