Bug 34657 - Add additional parameter to create_rvct_stubs for setting the offset
Summary: Add additional parameter to create_rvct_stubs for setting the offset
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-02-05 12:39 PST by Patrick R. Gansterer
Modified: 2010-02-15 10:05 PST (History)
4 users (show)

See Also:


Attachments
The patch (2.67 KB, patch)
2010-02-05 12:40 PST, Patrick R. Gansterer
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick R. Gansterer 2010-02-05 12:39:10 PST
see patch
Comment 1 Patrick R. Gansterer 2010-02-05 12:40:14 PST
Created attachment 48251 [details]
The patch
Comment 2 Laszlo Gombos 2010-02-08 20:08:28 PST
Change looks good to me, but I'd like to have more information on the motivation.

Which WebKit port intended to use this new feature ? 

I assume this work is for USE(JSVALUE32_64) - can you confirm ? 

Any reason for the following change:

> -my $file = $ARGV[0];

> +my $file;
> +$file = $ARGV[0];
Comment 3 Gabor Loki 2010-02-09 00:12:55 PST
Currently, only the JavaScriptCore/DerivedSources.pro uses the create_rvct_stubs script, but you did not change that pro file. Although the patch looks good, it is not complete. If you want to have this feature, please show an example how it will be used!
Comment 4 Patrick R. Gansterer 2010-02-13 14:30:52 PST
(In reply to comment #2)
> Change looks good to me, but I'd like to have more information on the
> motivation.
It's the first part for the WinCE JIT. Inline assabmler isn't supported and a additional file must be generated. The file should thenbe renamed to create_jit_stubs and have additional parameters for the "regex-prefix" (in the moment the RVCT([code])) and for skipping DEFINE_STUB_FUNCTION(int, op_eq_strings)

> Which WebKit port intended to use this new feature ?
WinCE

> I assume this work is for USE(JSVALUE32_64) - can you confirm ?
Of corse.

> Any reason for the following change:
No! ;-)

(In reply to comment #3)
> Currently, only the JavaScriptCore/DerivedSources.pro uses the
> create_rvct_stubs script, but you did not change that pro file. Although the
> patch looks good, it is not complete. If you want to have this feature, please
> show an example how it will be used!
I need it for the WinCE port where we don't have a buildsystem in the moment :-(
Comment 5 Gabor Loki 2010-02-13 21:11:18 PST
> > I assume this work is for USE(JSVALUE32_64) - can you confirm ?
> Of corse.
> ...
> I need it for the WinCE port where we don't have a buildsystem in the moment
> :-(

Ok, it is fine by me.

Btw, I would like to point on a problem of create_rvct_stubs. Currently the
Platform.h controls the usage of JSVALUE32_64. If Symbian and later WinCE
want to use JSVALUE32_64, their build systems will have to take over the
control JSVALUE32_64 from Platform.h (because of the generated stubs file).
Well, basically it is not a bug, it is an issue which has to be keep in mind.
Comment 6 Laszlo Gombos 2010-02-15 09:33:34 PST
Comment on attachment 48251 [details]
The patch

Thanks for the clarifications Patrick.

LGTM, r+.
Comment 7 WebKit Commit Bot 2010-02-15 10:05:02 PST
Comment on attachment 48251 [details]
The patch

Clearing flags on attachment: 48251

Committed r54781: <http://trac.webkit.org/changeset/54781>
Comment 8 WebKit Commit Bot 2010-02-15 10:05:11 PST
All reviewed patches have been landed.  Closing bug.