diff options
| author | Austin Seipp <aseipp@pobox.com> | 2013-09-09 10:42:55 -0500 |
|---|---|---|
| committer | Austin Seipp <aseipp@pobox.com> | 2013-09-09 10:44:57 -0500 |
| commit | 9f8e9d29c1c6e4e031469efcfbdae508dd6540e7 (patch) | |
| tree | 54170ceeee83bf4de08c6159a451a086b5ea4f01 /compiler/parser/Parser.y.pp | |
| parent | 9fb15842df6f3db7598b11843c33f761fb44ea87 (diff) | |
| download | haskell-9f8e9d29c1c6e4e031469efcfbdae508dd6540e7.tar.gz | |
Fix optimization of Parser.y.pp
In 9e133b, the build was modified to pass -fcmm-sink to Parser, but
unfortunately Parser specifies -O0 in its OPTIONS_GHC directive, meaning
the sinking pass was actually turned off.
HC_OPTS is the last thing passed to the compiler for that source file
however, so the correct fix is to also move -O0 out into the build
system as well.
This was uncovered thanks to a build report from Kazu Yamamoto. Thanks
to Jan Stolarek for eyeballing this bug and finding it.
Signed-off-by: Austin Seipp <aseipp@pobox.com>
Diffstat (limited to 'compiler/parser/Parser.y.pp')
| -rw-r--r-- | compiler/parser/Parser.y.pp | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/compiler/parser/Parser.y.pp b/compiler/parser/Parser.y.pp index b18d0d35c6..bd2bd0b922 100644 --- a/compiler/parser/Parser.y.pp +++ b/compiler/parser/Parser.y.pp @@ -16,14 +16,6 @@ -- http://hackage.haskell.org/trac/ghc/wiki/Commentary/CodingStyle#Warnings -- for details -{-# OPTIONS_GHC -O0 -fno-ignore-interface-pragmas #-} -{- -Careful optimisation of the parser: we don't want to throw everything -at it, because that takes too long and doesn't buy much, but we do want -to inline certain key external functions, so we instruct GHC not to -throw away inlinings as it would normally do in -O0 mode. --} - module Parser ( parseModule, parseStmt, parseIdentifier, parseType, parseHeader ) where |