[llvm] [BasicAA] Enable separate storage hints by default (PR #76864)

David Goldblatt via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 3 12:49:38 PST 2024


https://github.com/davidtgoldblatt created https://github.com/llvm/llvm-project/pull/76864

As requested in https://github.com/llvm/llvm-project/pull/76770#pullrequestreview-1801649466

A few months of experimentation in a large codebase did not reveal any significant build speed regressions, and b07bf16 speeds up hint lookup even further.

>From 42c7c0982047b195100bcecb5c0e5621de8071a9 Mon Sep 17 00:00:00 2001
From: David Goldblatt <davidgoldblatt at meta.com>
Date: Wed, 3 Jan 2024 12:44:21 -0800
Subject: [PATCH] [BasicAA] Enable separate storage hints by default

A few months of experimentation in a large codebase did not reveal any
significant build speed regressions, and b07bf16 speeds up hint lookup
even further.
---
 llvm/lib/Analysis/BasicAliasAnalysis.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/llvm/lib/Analysis/BasicAliasAnalysis.cpp b/llvm/lib/Analysis/BasicAliasAnalysis.cpp
index 4ad70931fd4cfd..97f60d28e49917 100644
--- a/llvm/lib/Analysis/BasicAliasAnalysis.cpp
+++ b/llvm/lib/Analysis/BasicAliasAnalysis.cpp
@@ -69,7 +69,7 @@ static cl::opt<bool> EnableRecPhiAnalysis("basic-aa-recphi", cl::Hidden,
                                           cl::init(true));
 
 static cl::opt<bool> EnableSeparateStorageAnalysis("basic-aa-separate-storage",
-                                                   cl::Hidden, cl::init(false));
+                                                   cl::Hidden, cl::init(true));
 
 /// SearchLimitReached / SearchTimes shows how often the limit of
 /// to decompose GEPs is reached. It will affect the precision



More information about the llvm-commits mailing list