[erlang-patches] Fix hibernate with HiPE

Paul Guyot pguyot@REDACTED
Sun Oct 10 20:01:43 CEST 2010


Le 10 oct. 2010 à 19:32, Mikael Pettersson a écrit :

> Looking at the patch via github's awful web interface it looks like
> it will cause the 'is_recursive' variable to be conditionally defined
> and possibly used uninitialized (at least as far as GCC will be able
> to tell).  Thus, NAK for now.
> 
> I would have provided the fix but you didn't include the patch in the
> email so I can't.

Mikael,

Thank you for your e-mail and review. While the is_recursive variable will currently not be used uninitialized (if stack is emptied by erts_garbage_collect_hibernate, status is set to P_WAITING), I agree this is a mistake and I have amended the patch accordingly. It is now available here:

http://github.com/pguyot/otp/commit/75e39faaa3180a78e8dab8d7bd4bef066dae579b

Paul
-- 
Semiocast                    http://semiocast.com/
+33.175000290 - 62 bis rue Gay-Lussac, 75005 Paris



More information about the erlang-patches mailing list