RESOLVED FIXED 137873
MacroAssemblerX86Common.cpp should be built on Windows too 
https://bugs.webkit.org/show_bug.cgi?id=137873
Summary MacroAssemblerX86Common.cpp should be built on Windows too
Csaba Osztrogonác
Reported 2014-10-20 03:43:28 PDT
MacroAssemblerX86Common.cpp should be built on Windows too
Attachments
Patch (2.54 KB, patch)
2014-10-20 03:43 PDT, Csaba Osztrogonác 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Csaba Osztrogonác
Comment 1 2014-10-20 03:43:51 PDT
Created attachment 240106 [details] Patch
Csaba Osztrogonác
Comment 2 2014-10-20 04:00:26 PDT
The goal of this bug report to add MacroAssemblerX86Common.cpp properly to the Windows build system to fix the build in bug137807 . I think ClInclude for this cpp was only a typo, when this file was added to the trunk - http://trac.webkit.org/changeset/153162/trunk/Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj Additionally it was added to the filter file as ClCompile - http://trac.webkit.org/changeset/153162/trunk/Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters But it was changed back to ClInclude in http://trac.webkit.org/changeset/153343/trunk/Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters
Brent Fulgham
Comment 3 2014-10-20 09:24:36 PDT
Comment on attachment 240106 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=240106&action=review Thanks for catching this. r=me. > Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj:308 > + <ClCompile Include="..\assembler\MacroAssemblerX86Common.cpp" /> This was definitely a mistake made by someone manually copying/pasting the project file.
WebKit Commit Bot
Comment 4 2014-10-20 10:03:20 PDT
Comment on attachment 240106 [details] Patch Clearing flags on attachment: 240106 Committed r174887: <http://trac.webkit.org/changeset/174887>
WebKit Commit Bot
Comment 5 2014-10-20 10:03:24 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.