RESOLVED FIXED 121250
GPRTemporary's reuse constructor should be templatized to reduce code duplication, and the bool to denote tag or payload should be replaced with an enum 
https://bugs.webkit.org/show_bug.cgi?id=121250
Summary GPRTemporary's reuse constructor should be templatized to reduce code duplica...
Filip Pizlo
Reported 2013-09-12 13:01:26 PDT
Patch forthcoming.
Attachments
the patch (36.85 KB, patch)
2013-09-12 13:02 PDT, Filip Pizlo 		oliver: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Filip Pizlo
Comment 1 2013-09-12 13:02:06 PDT
Created attachment 211462 [details] the patch
Filip Pizlo
Comment 2 2013-09-12 13:06:23 PDT
Landed in http://trac.webkit.org/changeset/155643
Ryosuke Niwa
Comment 3 2013-09-12 14:08:36 PDT
Qt build fixed in http://trac.webkit.org/changeset/155649 for /ramdisk/qt-linux-release/build/Source/JavaScriptCore/dfg/DFGGPRInfo.h: In member function 'JSC::DFG::GPRReg JSC::DFG::JSValueRegs::gpr(JSC::WhichValueWord) const': [1;31m/ramdisk/qt-linux-release/build/Source/JavaScriptCore/dfg/DFGGPRInfo.h:169:5: error: control reaches end of non-void function [-Werror=return-type][0m cc1plus: all warnings being treated as errors I've sent out a complaint about the warning. It's totally bogus.
Note You need to log in before you can comment on or make changes to this bug.