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

James Molloy via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 12 08:22:44 PST 2015


jmolloy updated this revision to Diff 40057.
jmolloy added a comment.

Hi Manman, Mehdi,

Updated patch attached.

- Now, Clang knows how to add norecurse to "main", so we can remove that logic (hooray!)
- Removed all the norecurse logic, replaced with a query to F->doesNotRecurse()

Please take a look.

Cheers,

James


Repository:
  rL LLVM

http://reviews.llvm.org/D14148

Files:
  lib/Transforms/IPO/GlobalOpt.cpp
  test/Transforms/GlobalOpt/global-demotion.ll
  test/Transforms/GlobalOpt/metadata.ll
  test/Transforms/MergeFunc/crash2.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D14148.40057.patch
Type: text/x-patch
Size: 11866 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151112/f0eb93bc/attachment.bin>


More information about the llvm-commits mailing list