Closed
Bug 655209
Opened 13 years ago
Closed 13 years ago
Sparc build broken / Makefile not enabling arch for methodjit files
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: andrew, Assigned: andrew)
Details
(Whiteboard: wanted-standalone-js)
Attachments
(1 file, 1 obsolete file)
702 bytes,
patch
|
ted
:
review-
|
Details | Diff | Splinter Review |
Now that the Sparc methodjit support has landed on m-c, the build is broken because ENABLE_METHODJIT and ENABLE_ASSEMBLER are defined, but the Makefile does not include 'sparc' in the list of supported architectures. The assembler files are not included in the .a file and the link fails. This patch simply adds 'sparc' to the list of archs in the Makefile so that the build works. Confirmed this allows my js shell to link.
Assignee | ||
Comment 1•13 years ago
|
||
Attachment #530553 -
Attachment is obsolete: true
Attachment #530555 -
Flags: review?(ted.mielczarek)
Updated•13 years ago
|
Assignee: general → andrew
Comment 2•13 years ago
|
||
Comment on attachment 530555 [details] [diff] [review] Add sparc arch to list of methodjit archs Review of attachment 530555 [details] [diff] [review]: ----------------------------------------------------------------- ::: js/src/Makefile.in @@ +415,4 @@ > # END enclude sources for V8 dtoa > ############################################# > > +ifeq (,$(filter arm% %86 x86_64 sparc,$(TARGET_CPU))) This is getting unbearable. Should this just be ENABLE_METHODJIT or something like that? @@ +472,5 @@ > ifeq (arm, $(TARGET_CPU)) > #CPPSRCS += only_on_arm.cpp > endif > +ifeq (sparc, $(TARGET_CPU)) > +#CPPSRCS += only_on_sparc.cpp What's the point of this commented-out bit? Are you just cargo-culting from above? I don't think we should bother having useless blocks here.
Attachment #530555 -
Flags: review?(ted.mielczarek) → review-
Comment 3•13 years ago
|
||
I conferred with pbiggar and he thinks you can just replace that ugly filter line with ifdef ENABLE_METHODJIT
Updated•13 years ago
|
Whiteboard: wanted-standalone-js
Assignee | ||
Comment 4•13 years ago
|
||
This was changed to ENABLE_METHOJIT in some other rev.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•