110565 – Renaming the armv7.rb LLINT backend to arm.rb

Bug 110565 - Renaming the armv7.rb LLINT backend to arm.rb

Summary: Renaming the armv7.rb LLINT backend to arm.rb

Status:	RESOLVED FIXED

Alias:	None

Product:	WebKit
Classification:	Unclassified
Component:	JavaScriptCore (show other bugs)
Version:	528+ (Nightly build)
Hardware:	Unspecified Unspecified

Importance:	P2 Normal
Assignee:	Nobody

URL:
Keywords:

Depends on:
Blocks:	97589
	Show dependency tree / graph

Reported:	2013-02-22 01:00 PST by Gabor Rapcsanyi
Modified:	2013-03-12 02:35 PDT (History)
CC List:	8 users (show)

See Also:

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 an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Gabor Rapcsanyi 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.

Comment 1 Gabor Rapcsanyi 2013-02-22 04:03:03 PST

Created attachment 189743 [details]
proposed patch

Comment 2 Gabor Rapcsanyi 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

Comment 3 Csaba Osztrogonác 2013-03-05 08:02:26 PST

ping for review, please :)

Comment 4 Zoltan Herczeg 2013-03-06 04:01:55 PST

This patch looks sane for me, so if noone has any objections, I will give an r+.

Comment 5 Zoltan Herczeg 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.

Comment 6 Gabor Rapcsanyi 2013-03-12 02:35:42 PDT

Thanks for the review!
Patch landed: http://trac.webkit.org/changeset/145504