Summary: | webcore should depend on webcore_arm_neon for only arm target_arch | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Wei James (wistoch) <james.wei> | ||||
Component: | WebCore Misc. | Assignee: | Wei James (wistoch) <james.wei> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | abarth, thakis, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Wei James (wistoch)
2012-06-11 22:51:30 PDT
Created attachment 147011 [details]
Patch
Why do you need this? Since https://bugs.webkit.org/show_bug.cgi?id=80626 , webcore_arm_neon is an empty target on non-arm, and depending on that should be harmless. (In reply to comment #2) > Why do you need this? Since https://bugs.webkit.org/show_bug.cgi?id=80626 , webcore_arm_neon is an empty target on non-arm, and depending on that should be harmless. yes, it is harmless. But it is very strange I found webcore_arm_neon.stamp in somewhere when I was building other targets. I'm fine with this landing, but I'm not a webkit reviewer, so I can't r+ (In reply to comment #4) > I'm fine with this landing, but I'm not a webkit reviewer, so I can't r+ +abarth. could you kindly help to review it? thanks Comment on attachment 147011 [details]
Patch
ok
Comment on attachment 147011 [details] Patch Clearing flags on attachment: 147011 Committed r120160: <http://trac.webkit.org/changeset/120160> All reviewed patches have been landed. Closing bug. |