[PATCH] D14148: [GlobalOpt] Demote globals to locals more aggressively

Mehdi Amini via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 16 09:46:50 PST 2015


> On Nov 16, 2015, at 9:44 AM, James Molloy <James.Molloy at arm.com> wrote:
> 
> 
>> On 16 Nov 2015, at 17:35, Duncan P. N. Exon Smith <dexonsmith at apple.com <mailto:dexonsmith at apple.com>> wrote:
>> 
>> Does this cause any benchmark regressions?  If not then maybe we don't
>> need to solve this.
> 
> None that I’ve observed, but of course I don’t know what the code was originally intended to improve.


Did you run the test suite with LTO? 
I know there are cases sensitive to this optimization.

— 
Mehdi

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151116/18a1052a/attachment.html>


More information about the llvm-commits mailing list