Bug 72212 - [CMake] Move list of DFG source files into correct file
: [CMake] Move list of DFG source files into correct file
Status: RESOLVED FIXED
: WebKit
Tools / Tests
: 528+ (Nightly build)
: All All
: P2 Normal
Assigned To:
:
:
:
:
  Show dependency treegraph
 
Reported: 2011-11-12 06:15 PST by
Modified: 2011-11-12 09:31 PST (History)


Attachments
Patch (2.86 KB, patch)
2011-11-12 06:20 PST, Patrick R. Gansterer
dbates: review+
dbates: commit‑queue-
Review Patch | Details | Formatted Diff | Diff


Note

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


Description From 2011-11-12 06:15:28 PST
[CMake] Move list of DFG source files into correct file
------- Comment #1 From 2011-11-12 06:20:44 PST -------
Created an attachment (id=114831) [details]
Patch
------- Comment #2 From 2011-11-12 08:53:14 PST -------
(From update of attachment 114831 [details])
View in context: https://bugs.webkit.org/attachment.cgi?id=114831&action=review

> Source/JavaScriptCore/CMakeLists.txt:57
> +    dfg/DFGJITCodeGenerator64.cpp
> +    dfg/DFGJITCodeGenerator32_64.cpp

These entries should be reversed so that they appear in sorted order as produced by the Unix sort command.

> Source/JavaScriptCore/CMakeLists.txt:59
> +    dfg/DFGOperations.cpp

This entry should be above dfg/DFGPropagator.cpp so that it appears in this list in sorted order as produced by the Unix sort command.

> Source/JavaScriptCore/CMakeLists.txt:64
> +    dfg/DFGOSRExitCompiler64.cpp
> +    dfg/DFGOSRExitCompiler32_64.cpp

These entries should be reversed so that they appear in sorted order as produced by the Unix sort command.

> Source/JavaScriptCore/CMakeLists.txt:69
> +    dfg/DFGSpeculativeJIT64.cpp
> +    dfg/DFGSpeculativeJIT32_64.cpp

Ditto.
------- Comment #3 From 2011-11-12 08:54:05 PST -------
(From update of attachment 114831 [details])
View in context: https://bugs.webkit.org/attachment.cgi?id=114831&action=review

> Source/JavaScriptCore/ChangeLog:8
> +        The DFG files are platform independet. So move them from

independet => independent
------- Comment #4 From 2011-11-12 09:31:17 PST -------
Committed r100081: <http://trac.webkit.org/changeset/100081>