On Apr 28, 2011, at 5:25 PM, Jakob Stoklund Olesen wrote: > > On Apr 28, 2011, at 3:19 PM, Chad Rosier wrote: > >> Attached is a patch (against trunk r130433) to add support for the -Oz >> command line option. The -Oz option lowers the inlining threshold to 25. > > Would it make sense to lower the threshold all the way to 0 for -Oz? The > InlineCost can actually become negative when it looks like inlining will > decrease code size, so that wouldn't disable inlining completely.
The value of 25 was chosen to match that in llvm-gcc. Further performance studies are necessary to determine what would be considered an "optimal" value. For now the value remains at 25. > /jakob > _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
