Slaying a dragon, the old-school way
Ok, the following isn't necessarily about making your code more efficient (in the sense it executes faster), just a couple of quick observations/pointers. 


Duplicate code like your two identical report() functions should
be avoided.
Your if..else if..else if statement may be better suited as a switch()/case.  Under some conditions, switch() can be more efficient than if/else-if/.. (depending on the javascript engine, etc.)   I should note that developers have been fighting the "switch-vs-if-else-if war for a while now.  https://stackoverflow.com/questions/2922948/javascript-switch-vs-if-else-if-else
Your dice values appear fixed (ie. they do not change).  Perhaps this is as intended, but I would suggest a rollDice() function is missing.
Closures (nested functions, or "functions within functions" if you like) may be a difficult concept for beginners...  It can introduce problems with variable scope that can be difficult to understand/deal with.   Disclaimer:  I am not saying closures are bad, they are powerful - but perhaps overkill for your current purposes, that's all.