2 % (c) The AQUA Project, Glasgow University, 1994-1998
4 \section[CoreUnfold]{Core-syntax unfoldings}
6 Unfoldings (which can travel across module boundaries) are in Core
7 syntax (namely @CoreExpr@s).
9 The type @Unfolding@ sits ``above'' simply-Core-expressions
10 unfoldings, capturing ``higher-level'' things we know about a binding,
11 usually things that the simplifier found out (e.g., ``it's a
12 literal''). In the corner of a @CoreUnfolding@ unfolding, you will
13 find, unsurprisingly, a Core expression.
17 Unfolding, UnfoldingGuidance, -- Abstract types
19 noUnfolding, mkTopUnfolding, mkUnfolding, mkCompulsoryUnfolding, seqUnfolding,
20 mkOtherCon, otherCons,
21 unfoldingTemplate, maybeUnfoldingTemplate,
22 isEvaldUnfolding, isValueUnfolding, isCheapUnfolding, isCompulsoryUnfolding,
23 hasUnfolding, hasSomeUnfolding, neverUnfold,
25 couldBeSmallEnoughToInline,
32 #include "HsVersions.h"
34 import CmdLineOpts ( opt_UF_CreationThreshold,
36 opt_UF_FunAppDiscount,
38 opt_UF_DearOp, opt_UnfoldCasms,
39 DynFlags, DynFlag(..), dopt
42 import PprCore ( pprCoreExpr )
43 import OccurAnal ( occurAnalyseGlobalExpr )
44 import CoreUtils ( exprIsValue, exprIsCheap, exprIsTrivial )
45 import Id ( Id, idType, isId,
47 isFCallId_maybe, globalIdDetails
49 import DataCon ( isUnboxedTupleCon )
50 import Literal ( isLitLitLit, litSize )
51 import PrimOp ( primOpIsDupable, primOpOutOfLine )
52 import ForeignCall ( okToExposeFCall )
53 import IdInfo ( OccInfo(..), GlobalIdDetails(..) )
54 import Type ( isUnLiftedType )
55 import PrelNames ( hasKey, buildIdKey, augmentIdKey )
60 #if __GLASGOW_HASKELL__ >= 404
61 import GlaExts ( fromInt, Int# )
66 %************************************************************************
68 \subsection{Making unfoldings}
70 %************************************************************************
73 mkTopUnfolding expr = mkUnfolding True {- Top level -} expr
75 mkUnfolding top_lvl expr
76 = CoreUnfolding (occurAnalyseGlobalExpr expr)
83 -- OK to inline inside a lambda
85 (calcUnfoldingGuidance opt_UF_CreationThreshold expr)
86 -- Sometimes during simplification, there's a large let-bound thing
87 -- which has been substituted, and so is now dead; so 'expr' contains
88 -- two copies of the thing while the occurrence-analysed expression doesn't
89 -- Nevertheless, we don't occ-analyse before computing the size because the
90 -- size computation bales out after a while, whereas occurrence analysis does not.
92 -- This can occasionally mean that the guidance is very pessimistic;
93 -- it gets fixed up next round
95 mkCompulsoryUnfolding expr -- Used for things that absolutely must be unfolded
96 = CompulsoryUnfolding (occurAnalyseGlobalExpr expr)
100 %************************************************************************
102 \subsection{The UnfoldingGuidance type}
104 %************************************************************************
107 instance Outputable UnfoldingGuidance where
108 ppr UnfoldNever = ptext SLIT("NEVER")
109 ppr (UnfoldIfGoodArgs v cs size discount)
110 = hsep [ ptext SLIT("IF_ARGS"), int v,
111 brackets (hsep (map int cs)),
118 calcUnfoldingGuidance
119 :: Int -- bomb out if size gets bigger than this
120 -> CoreExpr -- expression to look at
122 calcUnfoldingGuidance bOMB_OUT_SIZE expr
123 = case collect_val_bndrs expr of { (inline, val_binders, body) ->
125 n_val_binders = length val_binders
127 max_inline_size = n_val_binders+2
128 -- The idea is that if there is an INLINE pragma (inline is True)
129 -- and there's a big body, we give a size of n_val_binders+2. This
130 -- This is just enough to fail the no-size-increase test in callSiteInline,
131 -- so that INLINE things don't get inlined into entirely boring contexts,
135 case (sizeExpr (iUnbox bOMB_OUT_SIZE) val_binders body) of
138 | not inline -> UnfoldNever
139 -- A big function with an INLINE pragma must
140 -- have an UnfoldIfGoodArgs guidance
141 | inline -> UnfoldIfGoodArgs n_val_binders
142 (map (const 0) val_binders)
145 SizeIs size cased_args scrut_discount
148 (map discount_for val_binders)
150 (iBox scrut_discount)
152 boxed_size = iBox size
154 final_size | inline = boxed_size `min` max_inline_size
155 | otherwise = boxed_size
157 -- Sometimes an INLINE thing is smaller than n_val_binders+2.
158 -- A particular case in point is a constructor, which has size 1.
159 -- We want to inline this regardless, hence the `min`
161 discount_for b = foldlBag (\acc (b',n) -> if b==b' then acc+n else acc)
165 collect_val_bndrs e = go False [] e
166 -- We need to be a bit careful about how we collect the
167 -- value binders. In ptic, if we see
168 -- __inline_me (\x y -> e)
169 -- We want to say "2 value binders". Why? So that
170 -- we take account of information given for the arguments
172 go inline rev_vbs (Note InlineMe e) = go True rev_vbs e
173 go inline rev_vbs (Lam b e) | isId b = go inline (b:rev_vbs) e
174 | otherwise = go inline rev_vbs e
175 go inline rev_vbs e = (inline, reverse rev_vbs, e)
179 sizeExpr :: Int# -- Bomb out if it gets bigger than this
180 -> [Id] -- Arguments; we're interested in which of these
185 sizeExpr bOMB_OUT_SIZE top_args expr
188 size_up (Type t) = sizeZero -- Types cost nothing
189 size_up (Var v) = sizeOne
191 size_up (Note InlineMe body) = sizeOne -- Inline notes make it look very small
192 -- This can be important. If you have an instance decl like this:
193 -- instance Foo a => Foo [a] where
194 -- {-# INLINE op1, op2 #-}
197 -- then we'll get a dfun which is a pair of two INLINE lambdas
199 size_up (Note _ body) = size_up body -- Other notes cost nothing
201 size_up (App fun (Type t)) = size_up fun
202 size_up (App fun arg) = size_up_app fun [arg]
204 size_up (Lit lit) = sizeN (litSize lit)
206 size_up (Lam b e) | isId b = lamScrutDiscount (size_up e `addSizeN` 1)
207 | otherwise = size_up e
209 size_up (Let (NonRec binder rhs) body)
210 = nukeScrutDiscount (size_up rhs) `addSize`
211 size_up body `addSizeN`
212 (if isUnLiftedType (idType binder) then 0 else 1)
213 -- For the allocation
214 -- If the binder has an unlifted type there is no allocation
216 size_up (Let (Rec pairs) body)
217 = nukeScrutDiscount rhs_size `addSize`
218 size_up body `addSizeN`
219 length pairs -- For the allocation
221 rhs_size = foldr (addSize . size_up . snd) sizeZero pairs
223 size_up (Case (Var v) _ alts)
224 | v `elem` top_args -- We are scrutinising an argument variable
226 {- I'm nuking this special case; BUT see the comment with case alternatives.
228 (a) It's too eager. We don't want to inline a wrapper into a
229 context with no benefit.
230 E.g. \ x. f (x+x) no point in inlining (+) here!
232 (b) It's ineffective. Once g's wrapper is inlined, its case-expressions
233 aren't scrutinising arguments any more
237 [alt] -> size_up_alt alt `addSize` SizeIs 0# (unitBag (v, 1)) 0#
238 -- We want to make wrapper-style evaluation look cheap, so that
239 -- when we inline a wrapper it doesn't make call site (much) bigger
240 -- Otherwise we get nasty phase ordering stuff:
243 -- If we inline g's wrapper, f looks big, and doesn't get inlined
244 -- into h; if we inline f first, while it looks small, then g's
245 -- wrapper will get inlined later anyway. To avoid this nasty
246 -- ordering difference, we make (case a of (x,y) -> ...),
247 -- *where a is one of the arguments* look free.
251 alts_size (foldr addSize sizeOne alt_sizes) -- The 1 is for the scrutinee
252 (foldr1 maxSize alt_sizes)
254 -- Good to inline if an arg is scrutinised, because
255 -- that may eliminate allocation in the caller
256 -- And it eliminates the case itself
259 alt_sizes = map size_up_alt alts
261 -- alts_size tries to compute a good discount for
262 -- the case when we are scrutinising an argument variable
263 alts_size (SizeIs tot tot_disc tot_scrut) -- Size of all alternatives
264 (SizeIs max max_disc max_scrut) -- Size of biggest alternative
265 = SizeIs tot (unitBag (v, iBox (_ILIT 1 +# tot -# max)) `unionBags` max_disc) max_scrut
266 -- If the variable is known, we produce a discount that
267 -- will take us back to 'max', the size of rh largest alternative
268 -- The 1+ is a little discount for reduced allocation in the caller
269 alts_size tot_size _ = tot_size
272 size_up (Case e _ alts) = nukeScrutDiscount (size_up e) `addSize`
273 foldr (addSize . size_up_alt) sizeZero alts
274 -- We don't charge for the case itself
275 -- It's a strict thing, and the price of the call
276 -- is paid by scrut. Also consider
277 -- case f x of DEFAULT -> e
278 -- This is just ';'! Don't charge for it.
281 size_up_app (App fun arg) args
282 | isTypeArg arg = size_up_app fun args
283 | otherwise = size_up_app fun (arg:args)
284 size_up_app fun args = foldr (addSize . nukeScrutDiscount . size_up)
285 (size_up_fun fun args)
288 -- A function application with at least one value argument
289 -- so if the function is an argument give it an arg-discount
291 -- Also behave specially if the function is a build
293 -- Also if the function is a constant Id (constr or primop)
294 -- compute discounts specially
295 size_up_fun (Var fun) args
296 | fun `hasKey` buildIdKey = buildSize
297 | fun `hasKey` augmentIdKey = augmentSize
299 = case globalIdDetails fun of
300 DataConId dc -> conSizeN dc (valArgCount args)
302 FCallId fc -> sizeN opt_UF_DearOp
303 PrimOpId op -> primOpSize op (valArgCount args)
304 -- foldr addSize (primOpSize op) (map arg_discount args)
305 -- At one time I tried giving an arg-discount if a primop
306 -- is applied to one of the function's arguments, but it's
307 -- not good. At the moment, any unlifted-type arg gets a
308 -- 'True' for 'yes I'm evald', so we collect the discount even
309 -- if we know nothing about it. And just having it in a primop
310 -- doesn't help at all if we don't know something more.
312 other -> fun_discount fun `addSizeN`
313 (1 + length (filter (not . exprIsTrivial) args))
314 -- The 1+ is for the function itself
315 -- Add 1 for each non-trivial arg;
316 -- the allocation cost, as in let(rec)
317 -- Slight hack here: for constructors the args are almost always
318 -- trivial; and for primops they are almost always prim typed
319 -- We should really only count for non-prim-typed args in the
320 -- general case, but that seems too much like hard work
322 size_up_fun other args = size_up other
325 size_up_alt (con, bndrs, rhs) = size_up rhs
326 -- Don't charge for args, so that wrappers look cheap
327 -- (See comments about wrappers with Case)
330 -- We want to record if we're case'ing, or applying, an argument
331 fun_discount v | v `elem` top_args = SizeIs 0# (unitBag (v, opt_UF_FunAppDiscount)) 0#
332 fun_discount other = sizeZero
335 -- These addSize things have to be here because
336 -- I don't want to give them bOMB_OUT_SIZE as an argument
338 addSizeN TooBig _ = TooBig
339 addSizeN (SizeIs n xs d) m = mkSizeIs bOMB_OUT_SIZE (n +# iUnbox m) xs d
341 addSize TooBig _ = TooBig
342 addSize _ TooBig = TooBig
343 addSize (SizeIs n1 xs d1) (SizeIs n2 ys d2)
344 = mkSizeIs bOMB_OUT_SIZE (n1 +# n2) (xs `unionBags` ys) (d1 +# d2)
347 Code for manipulating sizes
350 data ExprSize = TooBig
351 | SizeIs FastInt -- Size found
352 (Bag (Id,Int)) -- Arguments cased herein, and discount for each such
353 FastInt -- Size to subtract if result is scrutinised
354 -- by a case expression
356 mkSizeIs max n xs d | (n -# d) ># max = TooBig
357 | otherwise = SizeIs n xs d
359 maxSize TooBig _ = TooBig
360 maxSize _ TooBig = TooBig
361 maxSize s1@(SizeIs n1 _ _) s2@(SizeIs n2 _ _) | n1 ># n2 = s1
364 sizeZero = SizeIs (_ILIT 0) emptyBag (_ILIT 0)
365 sizeOne = SizeIs (_ILIT 1) emptyBag (_ILIT 0)
366 sizeN n = SizeIs (iUnbox n) emptyBag (_ILIT 0)
368 | isUnboxedTupleCon dc = SizeIs (_ILIT 0) emptyBag (iUnbox n +# _ILIT 1)
369 | otherwise = SizeIs (_ILIT 1) emptyBag (iUnbox n +# _ILIT 1)
370 -- Treat constructors as size 1; we are keen to expose them
371 -- (and we charge separately for their args). We can't treat
372 -- them as size zero, else we find that (iBox x) has size 1,
373 -- which is the same as a lone variable; and hence 'v' will
374 -- always be replaced by (iBox x), where v is bound to iBox x.
376 -- However, unboxed tuples count as size zero
377 -- I found occasions where we had
378 -- f x y z = case op# x y z of { s -> (# s, () #) }
379 -- and f wasn't getting inlined
382 | not (primOpIsDupable op) = sizeN opt_UF_DearOp
383 | not (primOpOutOfLine op) = sizeN (2 - n_args)
384 -- Be very keen to inline simple primops.
385 -- We give a discount of 1 for each arg so that (op# x y z) costs 2.
386 -- We can't make it cost 1, else we'll inline let v = (op# x y z)
387 -- at every use of v, which is excessive.
389 -- A good example is:
390 -- let x = +# p q in C {x}
391 -- Even though x get's an occurrence of 'many', its RHS looks cheap,
392 -- and there's a good chance it'll get inlined back into C's RHS. Urgh!
393 | otherwise = sizeOne
395 buildSize = SizeIs (-2#) emptyBag 4#
396 -- We really want to inline applications of build
397 -- build t (\cn -> e) should cost only the cost of e (because build will be inlined later)
398 -- Indeed, we should add a result_discount becuause build is
399 -- very like a constructor. We don't bother to check that the
400 -- build is saturated (it usually is). The "-2" discounts for the \c n,
401 -- The "4" is rather arbitrary.
403 augmentSize = SizeIs (-2#) emptyBag 4#
404 -- Ditto (augment t (\cn -> e) ys) should cost only the cost of
405 -- e plus ys. The -2 accounts for the \cn
407 nukeScrutDiscount (SizeIs n vs d) = SizeIs n vs 0#
408 nukeScrutDiscount TooBig = TooBig
410 -- When we return a lambda, give a discount if it's used (applied)
411 lamScrutDiscount (SizeIs n vs d) = case opt_UF_FunAppDiscount of { d -> SizeIs n vs (iUnbox d) }
412 lamScrutDiscount TooBig = TooBig
416 %************************************************************************
418 \subsection[considerUnfolding]{Given all the info, do (not) do the unfolding}
420 %************************************************************************
422 We have very limited information about an unfolding expression: (1)~so
423 many type arguments and so many value arguments expected---for our
424 purposes here, we assume we've got those. (2)~A ``size'' or ``cost,''
425 a single integer. (3)~An ``argument info'' vector. For this, what we
426 have at the moment is a Boolean per argument position that says, ``I
427 will look with great favour on an explicit constructor in this
428 position.'' (4)~The ``discount'' to subtract if the expression
429 is being scrutinised.
431 Assuming we have enough type- and value arguments (if not, we give up
432 immediately), then we see if the ``discounted size'' is below some
433 (semi-arbitrary) threshold. It works like this: for every argument
434 position where we're looking for a constructor AND WE HAVE ONE in our
435 hands, we get a (again, semi-arbitrary) discount [proportion to the
436 number of constructors in the type being scrutinized].
438 If we're in the context of a scrutinee ( \tr{(case <expr > of A .. -> ...;.. )})
439 and the expression in question will evaluate to a constructor, we use
440 the computed discount size *for the result only* rather than
441 computing the argument discounts. Since we know the result of
442 the expression is going to be taken apart, discounting its size
443 is more accurate (see @sizeExpr@ above for how this discount size
446 We use this one to avoid exporting inlinings that we ``couldn't possibly
447 use'' on the other side. Can be overridden w/ flaggery.
448 Just the same as smallEnoughToInline, except that it has no actual arguments.
451 couldBeSmallEnoughToInline :: Int -> CoreExpr -> Bool
452 couldBeSmallEnoughToInline threshold rhs = case calcUnfoldingGuidance threshold rhs of
456 certainlyWillInline :: Unfolding -> Bool
457 -- Sees if the unfolding is pretty certain to inline
458 certainlyWillInline (CoreUnfolding _ _ _ is_cheap (UnfoldIfGoodArgs n_vals _ size _))
459 = is_cheap && size - (n_vals +1) <= opt_UF_UseThreshold
460 certainlyWillInline other
464 @okToUnfoldInHifile@ is used when emitting unfolding info into an interface
465 file to determine whether an unfolding candidate really should be unfolded.
466 The predicate is needed to prevent @_casm_@s (+ lit-lits) from being emitted
467 into interface files.
469 The reason for inlining expressions containing _casm_s into interface files
470 is that these fragments of C are likely to mention functions/#defines that
471 will be out-of-scope when inlined into another module. This is not an
472 unfixable problem for the user (just need to -#include the approp. header
473 file), but turning it off seems to the simplest thing to do.
476 okToUnfoldInHiFile :: CoreExpr -> Bool
477 okToUnfoldInHiFile e = opt_UnfoldCasms || go e
479 -- Race over an expression looking for CCalls..
480 go (Var v) = case isFCallId_maybe v of
481 Just fcall -> okToExposeFCall fcall
483 go (Lit lit) = not (isLitLitLit lit)
484 go (App fun arg) = go fun && go arg
485 go (Lam _ body) = go body
486 go (Let binds body) = and (map go (body :rhssOfBind binds))
487 go (Case scrut bndr alts) = and (map go (scrut:rhssOfAlts alts)) &&
488 not (any isLitLitLit [ lit | (LitAlt lit, _, _) <- alts ])
489 go (Note _ body) = go body
494 %************************************************************************
496 \subsection{callSiteInline}
498 %************************************************************************
500 This is the key function. It decides whether to inline a variable at a call site
502 callSiteInline is used at call sites, so it is a bit more generous.
503 It's a very important function that embodies lots of heuristics.
504 A non-WHNF can be inlined if it doesn't occur inside a lambda,
505 and occurs exactly once or
506 occurs once in each branch of a case and is small
508 If the thing is in WHNF, there's no danger of duplicating work,
509 so we can inline if it occurs once, or is small
511 NOTE: we don't want to inline top-level functions that always diverge.
512 It just makes the code bigger. Tt turns out that the convenient way to prevent
513 them inlining is to give them a NOINLINE pragma, which we do in
514 StrictAnal.addStrictnessInfoToTopId
517 callSiteInline :: DynFlags
518 -> Bool -- True <=> the Id can be inlined
519 -> Bool -- 'inline' note at call site
522 -> [Bool] -- One for each value arg; True if it is interesting
523 -> Bool -- True <=> continuation is interesting
524 -> Maybe CoreExpr -- Unfolding, if any
527 callSiteInline dflags active_inline inline_call occ id arg_infos interesting_cont
528 = case idUnfolding id of {
529 NoUnfolding -> Nothing ;
530 OtherCon cs -> Nothing ;
532 CompulsoryUnfolding unf_template -> Just unf_template ;
533 -- CompulsoryUnfolding => there is no top-level binding
534 -- for these things, so we must inline it.
535 -- Only a couple of primop-like things have
536 -- compulsory unfoldings (see MkId.lhs).
537 -- We don't allow them to be inactive
539 CoreUnfolding unf_template is_top is_value is_cheap guidance ->
542 result | yes_or_no = Just unf_template
543 | otherwise = Nothing
545 n_val_args = length arg_infos
548 | not active_inline = False
549 | otherwise = case occ of
550 IAmDead -> pprTrace "callSiteInline: dead" (ppr id) False
551 IAmALoopBreaker -> False
552 OneOcc in_lam one_br -> (not in_lam || is_cheap) && consider_safe in_lam True one_br
553 NoOccInfo -> is_cheap && consider_safe True False False
555 consider_safe in_lam once once_in_one_branch
556 -- consider_safe decides whether it's a good idea to inline something,
557 -- given that there's no work-duplication issue (the caller checks that).
558 -- once_in_one_branch = True means there's a unique textual occurrence
562 -- Be very keen to inline something if this is its unique occurrence:
564 -- a) Inlining gives a good chance of eliminating the original
565 -- binding (and hence the allocation) for the thing.
566 -- (Provided it's not a top level binding, in which case the
567 -- allocation costs nothing.)
569 -- b) Inlining a function that is called only once exposes the
570 -- body function to the call site.
572 -- The only time we hold back is when substituting inside a lambda;
573 -- then if the context is totally uninteresting (not applied, not scrutinised)
574 -- there is no point in substituting because it might just increase allocation,
575 -- by allocating the function itself many times
577 -- Note: there used to be a '&& not top_level' in the guard above,
578 -- but that stopped us inlining top-level functions used only once,
580 = WARN( not is_top && not in_lam, ppr id )
581 -- If (not in_lam) && one_br then PreInlineUnconditionally
582 -- should have caught it, shouldn't it? Unless it's a top
584 not (null arg_infos) || interesting_cont
588 UnfoldNever -> False ;
589 UnfoldIfGoodArgs n_vals_wanted arg_discounts size res_discount
591 | enough_args && size <= (n_vals_wanted + 1)
592 -- Inline unconditionally if there no size increase
593 -- Size of call is n_vals_wanted (+1 for the function)
597 -> some_benefit && small_enough
600 some_benefit = or arg_infos || really_interesting_cont ||
601 (not is_top && (once || (n_vals_wanted > 0 && enough_args)))
602 -- If it occurs more than once, there must be something interesting
603 -- about some argument, or the result context, to make it worth inlining
605 -- If a function has a nested defn we also record some-benefit,
606 -- on the grounds that we are often able to eliminate the binding,
607 -- and hence the allocation, for the function altogether; this is good
608 -- for join points. But this only makes sense for *functions*;
609 -- inlining a constructor doesn't help allocation unless the result is
610 -- scrutinised. UNLESS the constructor occurs just once, albeit possibly
611 -- in multiple case branches. Then inlining it doesn't increase allocation,
612 -- but it does increase the chance that the constructor won't be allocated at all
613 -- in the branches that don't use it.
615 enough_args = n_val_args >= n_vals_wanted
616 really_interesting_cont | n_val_args < n_vals_wanted = False -- Too few args
617 | n_val_args == n_vals_wanted = interesting_cont
618 | otherwise = True -- Extra args
619 -- really_interesting_cont tells if the result of the
620 -- call is in an interesting context.
622 small_enough = (size - discount) <= opt_UF_UseThreshold
623 discount = computeDiscount n_vals_wanted arg_discounts res_discount
624 arg_infos really_interesting_cont
627 if dopt Opt_D_dump_inlinings dflags then
628 pprTrace "Considering inlining"
629 (ppr id <+> vcat [text "active:" <+> ppr active_inline,
630 text "occ info:" <+> ppr occ,
631 text "arg infos" <+> ppr arg_infos,
632 text "interesting continuation" <+> ppr interesting_cont,
633 text "is value:" <+> ppr is_value,
634 text "is cheap:" <+> ppr is_cheap,
635 text "guidance" <+> ppr guidance,
636 text "ANSWER =" <+> if yes_or_no then text "YES" else text "NO",
638 text "Unfolding =" <+> pprCoreExpr unf_template
645 computeDiscount :: Int -> [Int] -> Int -> [Bool] -> Bool -> Int
646 computeDiscount n_vals_wanted arg_discounts res_discount arg_infos result_used
647 -- We multiple the raw discounts (args_discount and result_discount)
648 -- ty opt_UnfoldingKeenessFactor because the former have to do with
649 -- *size* whereas the discounts imply that there's some extra
650 -- *efficiency* to be gained (e.g. beta reductions, case reductions)
653 -- we also discount 1 for each argument passed, because these will
654 -- reduce with the lambdas in the function (we count 1 for a lambda
656 = 1 + -- Discount of 1 because the result replaces the call
657 -- so we count 1 for the function itself
658 length (take n_vals_wanted arg_infos) +
659 -- Discount of 1 for each arg supplied, because the
660 -- result replaces the call
661 round (opt_UF_KeenessFactor *
662 fromInt (arg_discount + result_discount))
664 arg_discount = sum (zipWith mk_arg_discount arg_discounts arg_infos)
666 mk_arg_discount discount is_evald | is_evald = discount
669 -- Don't give a result discount unless there are enough args
670 result_discount | result_used = res_discount -- Over-applied, or case scrut