WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
110565
Renaming the armv7.rb LLINT backend to arm.rb
https://bugs.webkit.org/show_bug.cgi?id=110565
Summary
Renaming the armv7.rb LLINT backend to arm.rb
Gabor Rapcsanyi
Reported
2013-02-22 01:00:32 PST
This is the first part of making available LLINT for ARM traditional and armv5. First I want to rename everything from armv7 to arm. After that I can add the traditional and armv5 specific stuff to it.
Attachments
proposed patch
(30.90 KB, patch)
2013-02-22 04:03 PST
,
Gabor Rapcsanyi
zherczeg
: review+
zherczeg
: commit-queue-
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Gabor Rapcsanyi
Comment 1
2013-02-22 04:03:03 PST
Created
attachment 189743
[details]
proposed patch
Gabor Rapcsanyi
Comment 2
2013-02-26 06:19:00 PST
Filip could you take a look at this please? The following patch would be this:
https://bugs.webkit.org/show_bug.cgi?id=97589
Csaba Osztrogonác
Comment 3
2013-03-05 08:02:26 PST
ping for review, please :)
Zoltan Herczeg
Comment 4
2013-03-06 04:01:55 PST
This patch looks sane for me, so if noone has any objections, I will give an r+.
Zoltan Herczeg
Comment 5
2013-03-12 02:17:26 PDT
Comment on
attachment 189743
[details]
proposed patch There was no objections, so it is time for a review. The patch is quite old now, so please update it before land. View in context:
https://bugs.webkit.org/attachment.cgi?id=189743&action=review
> Source/JavaScriptCore/ChangeLog:9 > + The first step to making a common ARM backend in LLINT > + for ARMv5, ARMv7 and ARMv7 traditional.
A little more details please.
Gabor Rapcsanyi
Comment 6
2013-03-12 02:35:42 PDT
Thanks for the review! Patch landed:
http://trac.webkit.org/changeset/145504
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug