That method name is already used, and is intended entirely for debug printing. It should never be used for code generation.
Created attachment 185339 [details] the patch
LGTM. Would feel more sure if you can set ENABLE_LLINT=1 ENABLE_JIT=0 in Platform.h and build to make sure it still builds and runs, but I expect it to be fine.
Comment on attachment 185339 [details] the patch r=me
> Would feel more sure if you can set ENABLE_LLINT=1 ENABLE_JIT=0 in Platform.h Actually, I believe this build is currently broken for other reasons :(.
(In reply to comment #4) > > Would feel more sure if you can set ENABLE_LLINT=1 ENABLE_JIT=0 in Platform.h > > Actually, I believe this build is currently broken for other reasons :(. Sad. =(
(In reply to comment #5) > (In reply to comment #4) > > > Would feel more sure if you can set ENABLE_LLINT=1 ENABLE_JIT=0 in Platform.h > > > > Actually, I believe this build is currently broken for other reasons :(. > > Sad. =( I fixed the build and tested my change. It's all good.
Landed in http://trac.webkit.org/changeset/141185
(In reply to comment #6) > (In reply to comment #5) > > (In reply to comment #4) > > > > Would feel more sure if you can set ENABLE_LLINT=1 ENABLE_JIT=0 in Platform.h > > > > > > Actually, I believe this build is currently broken for other reasons :(. > > > > Sad. =( > > I fixed the build and tested my change. It's all good. Yay! Thank you. =)