Skip to content

Commit 13eea8c

Browse files
committed
Remove INLINE pragma on timeOfDay64
timeOfDay64 is an enormous function, and marking it as INLINE causes a simple aeson benchmark to exhaust its simplifier ticks. Fixes #502.
1 parent 2abacb9 commit 13eea8c

File tree

2 files changed

+1
-2
lines changed

2 files changed

+1
-2
lines changed

Data/Aeson/Encoding/Builder.hs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -234,7 +234,6 @@ timeOfDay64 (TOD h m s)
234234
pico = 1000000000000 -- number of picoseconds in 1 second
235235
micro = 1000000 -- number of microseconds in 1 second
236236
milli = 1000 -- number of milliseconds in 1 second
237-
{-# INLINE timeOfDay64 #-}
238237

239238
timeZone :: TimeZone -> Builder
240239
timeZone (TimeZone off _ _)

benchmarks/aeson-benchmarks.cabal

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ executable aeson-benchmark-json-parse
179179

180180
executable aeson-benchmark-dates
181181
main-is: Dates.hs
182-
ghc-options: -Wall -O2 -rtsopts -fsimpl-tick-factor=200
182+
ghc-options: -Wall -O2 -rtsopts
183183
build-depends:
184184
base,
185185
base-compat,

0 commit comments

Comments
 (0)