RESOLVED FIXED 174297
[JSC] Use fastMalloc / fastFree for STL containers 
https://bugs.webkit.org/show_bug.cgi?id=174297
Summary [JSC] Use fastMalloc / fastFree for STL containers
Yusuke Suzuki
Reported 2017-07-09 02:02:13 PDT
[JSC] Use fastMalloc / fastFree for std::unordered_set / std::unordered_map
Attachments
Patch (8.55 KB, patch)
2017-07-09 02:10 PDT, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Patch (8.55 KB, patch)
2017-07-09 02:11 PDT, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Patch (8.56 KB, patch)
2017-07-09 02:23 PDT, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Patch (9.49 KB, patch)
2017-07-09 02:33 PDT, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Patch (8.58 KB, patch)
2017-07-09 02:38 PDT, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.
Yusuke Suzuki
Comment 1 2017-07-09 02:10:30 PDT
Created attachment 314943 [details] Patch
Yusuke Suzuki
Comment 2 2017-07-09 02:11:37 PDT
Created attachment 314944 [details] Patch
Yusuke Suzuki
Comment 3 2017-07-09 02:13:12 PDT
Comment on attachment 314944 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=314944&action=review > Source/WTF/wtf/FastMalloc.h:108 > +// e.g. std::unordered_map<Key, Value, std::hash<Key>, std::equal_to<Key>, FastAllocator<std::pair<const Key, Value>>>. C++11 std::allocator_traits significantly simplifies C++11 allocator requirement.
Yusuke Suzuki
Comment 4 2017-07-09 02:23:47 PDT
Created attachment 314945 [details] Patch
Yusuke Suzuki
Comment 5 2017-07-09 02:33:44 PDT
Created attachment 314946 [details] Patch
Yusuke Suzuki
Comment 6 2017-07-09 02:38:18 PDT
Created attachment 314947 [details] Patch
Yusuke Suzuki
Comment 7 2017-07-09 04:55:01 PDT
Comment on attachment 314947 [details] Patch Thanks!
WebKit Commit Bot
Comment 8 2017-07-09 05:23:02 PDT
Comment on attachment 314947 [details] Patch Clearing flags on attachment: 314947 Committed r219281: <http://trac.webkit.org/changeset/219281>
WebKit Commit Bot
Comment 9 2017-07-09 05:23:03 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.