Macro Expansion: Special case QuoteNode by Returnsing the unpacked value rather than the QuoteNode itself.

[LilithHafner]

Fixes #99.

This special case assumes that all non-Expr objects in an expression tree evaluate to themselves. This is a false assumption in the case of QuoteNode(::Symbol).

Additionally, this Returns gives the compiler slightly less information by inhibiting constant propagation of literals. Constant propagation of literals is usually bad (misleadingly low runtimes) in this case but this particular fix also results in @b 2 rand and @b 1+1 rand having different properties with the latter more constprop-able. I was not able to find a case where the Returns is helpful.

Returns was added in 4f8a8c8 to avoid compilation time and paired with the test

@testset "no compilation" begin
    res = @b @eval (@b 100 rand seconds=.001)
    @test .001 < 1e-9res.time < .002
    @test res.compile_fraction === 0.0
end

I expect the "no compilation" regression test to regress.

[LilithHafner]

The regression tests failed as expected. See comment in source code and commit comment of 9ec4ef5 for explanation. The OP is out of date.