Comment on attachment 632053 Workaround which only affects DEBUG builds
Review of attachment 632053: -----------------------------------------------------------------
::: js/public/Vector.h @@ +565,5 @@ > STATIC_POSTCONDITION(!return || newCap >= curLength + lengthInc) > +#ifdef DEBUG > +// By making this method not inline for debug builds, it sidesteps > +// the compiler generating incorrect code. See bug 694594. > +JS_NEVER_INLINE bool
Surrounding comments use /* */. Also, I think you could say, more succinctly:
/* ARM compiler bug workaround, see bug 694594. */
Comment on attachment 632053
Workaround which only affects DEBUG builds
Review of attachment 632053: ------- ------- ------- ------- ------- ------- ------- ------- --
-------
::: js/public/Vector.h POSTCONDITION( !return || newCap >= curLength + lengthInc)
@@ +565,5 @@
> STATIC_
> +#ifdef DEBUG
> +// By making this method not inline for debug builds, it sidesteps
> +// the compiler generating incorrect code. See bug 694594.
> +JS_NEVER_INLINE bool
Surrounding comments use /* */. Also, I think you could say, more succinctly:
/* ARM compiler bug workaround, see bug 694594. */