[llvm] [ObjectSizeOffsetVisitor] Add a max recursion depth (PR #67479)

Arthur Eubanks via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 26 13:20:01 PDT 2023


================
@@ -50,6 +51,12 @@ using namespace llvm;
 
 #define DEBUG_TYPE "memory-builtins"
 
+static cl::opt<unsigned> ObjectSizeOffsetVisitorMaxRecurseDepth(
+    "object-size-offset-visitor-max-recurse-depth",
+    cl::desc(
+        "Maximum number of PHIs for ObjectSizeOffsetVisitor to look through"),
+    cl::init(100));
----------------
aeubanks wrote:

Lowered to 20.
I think it's unlikely that we hit huge fanout in real world code, it's typically pretty linear at least with large autogenerated code. In the case I was looking at, this dropped DSE time from >30s to <1s.

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


More information about the llvm-commits mailing list