Bug 227377 - Remove references to order files
Summary: Remove references to order files
Status: REOPENED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Sihui Liu
URL:
Keywords: InRadar
Depends on: 227402
Blocks:
  Show dependency treegraph
 
Reported: 2021-06-24 14:21 PDT by Sihui Liu
Modified: 2021-06-25 08:04 PDT (History)
9 users (show)

See Also:


Attachments
Patch (6.47 KB, patch)
2021-06-24 14:26 PDT, Sihui Liu
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Sihui Liu 2021-06-24 14:21:42 PDT
rdar://76070556
Comment 1 Sihui Liu 2021-06-24 14:26:33 PDT
Created attachment 432215 [details]
Patch
Comment 2 Mark Lam 2021-06-24 14:30:03 PDT
Comment on attachment 432215 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=432215&action=review

> Source/JavaScriptCore/Configurations/JavaScriptCore.xcconfig:37
>  SECTORDER_FLAGS_Production[sdk=macosx*] = -Wl,-order_file,JavaScriptCore.order;

Do we need it for Mac?  Or can we purge those as well?
Comment 3 Sihui Liu 2021-06-24 14:43:48 PDT
(In reply to Mark Lam from comment #2)
> Comment on attachment 432215 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=432215&action=review
> 
> > Source/JavaScriptCore/Configurations/JavaScriptCore.xcconfig:37
> >  SECTORDER_FLAGS_Production[sdk=macosx*] = -Wl,-order_file,JavaScriptCore.order;
> 
> Do we need it for Mac?  Or can we purge those as well?

This (rdar://76070556) is only available on iOS now.
Comment 4 Mark Lam 2021-06-24 14:53:26 PDT
Comment on attachment 432215 [details]
Patch

r=me if EWS bots are green.
Comment 5 EWS 2021-06-24 20:30:07 PDT
Committed r279266 (239145@main): <https://commits.webkit.org/239145@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 432215 [details].
Comment 6 WebKit Commit Bot 2021-06-25 08:04:12 PDT
Re-opened since this is blocked by bug 227402