| Summary: | Do some CoW cleanup | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Saam Barati <saam> | ||||
| Component: | JavaScriptCore | Assignee: | Saam Barati <saam> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | benjamin, bugs-noreply, calvaris, clopez, commit-queue, fpizlo, ggaren, gskachkov, jfbastien, keith_miller, mark.lam, msaboff, rmorisset, ticaiolima, webkit-bug-importer, ysuzuki | ||||
| Priority: | P2 | Keywords: | InRadar | ||||
| Version: | WebKit Nightly Build | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Bug Depends on: | 187051 | ||||||
| Bug Blocks: | |||||||
| Attachments: |
|
||||||
|
Description
Saam Barati
2018-06-21 14:24:08 PDT
Created attachment 343278 [details]
patch
Comment on attachment 343278 [details]
patch
r=me
Comment on attachment 343278 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=343278&action=review > Source/JavaScriptCore/dfg/DFGByteCodeParser.cpp:-4598 > - // TODO: Do I need this? Whoops, that's definitely my bad! lol Comment on attachment 343278 [details] patch Clearing flags on attachment: 343278 Committed r233065: <https://trac.webkit.org/changeset/233065> All reviewed patches have been landed. Closing bug. (In reply to WebKit Commit Bot from comment #4) > Comment on attachment 343278 [details] > patch > > Clearing flags on attachment: 343278 > > Committed r233065: <https://trac.webkit.org/changeset/233065> This broke our Ubuntu bot (clang-3.8 with libstcd++-5.4) ./../Source/JavaScriptCore/bytecode/UnlinkedCodeBlock.h:312:16: error: chosen constructor is explicit in copy-initialization return { profile, recommendedIndexingType }; ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ /usr/bin/../lib/gcc/x86_64-linux-gnu/5.4.0/../../../../include/c++/5.4.0/tuple:612:19: note: constructor declared here constexpr tuple(_U1&& __a1, _U2&& __a2) ^ 1 error generated. Log: https://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release%20Ubuntu%20LTS%20%28Build%29/builds/13359/steps/compile-webkit/logs/stdio (In reply to Carlos Alberto Lopez Perez from comment #7) > (In reply to WebKit Commit Bot from comment #4) > > Comment on attachment 343278 [details] > > patch > > > > Clearing flags on attachment: 343278 > > > > Committed r233065: <https://trac.webkit.org/changeset/233065> > > This broke our Ubuntu bot (clang-3.8 with libstcd++-5.4) > > > ./../Source/JavaScriptCore/bytecode/UnlinkedCodeBlock.h:312:16: error: > chosen constructor is explicit in copy-initialization > return { profile, recommendedIndexingType }; > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > /usr/bin/../lib/gcc/x86_64-linux-gnu/5.4.0/../../../../include/c++/5.4.0/ > tuple:612:19: note: constructor declared here > constexpr tuple(_U1&& __a1, _U2&& __a2) > ^ > 1 error generated. > > > Log: > https://build.webkit.org/builders/GTK%20Linux%2064- > bit%20Release%20Ubuntu%20LTS%20%28Build%29/builds/13359/steps/compile-webkit/ > logs/stdio There probably just needs to be a WTFMove around profile and recommendedIndexingType. (In reply to Keith Miller from comment #8) > (In reply to Carlos Alberto Lopez Perez from comment #7) > > (In reply to WebKit Commit Bot from comment #4) > > > Comment on attachment 343278 [details] > > > patch > > > > > > Clearing flags on attachment: 343278 > > > > > > Committed r233065: <https://trac.webkit.org/changeset/233065> > > > > This broke our Ubuntu bot (clang-3.8 with libstcd++-5.4) > > > > > > ./../Source/JavaScriptCore/bytecode/UnlinkedCodeBlock.h:312:16: error: > > chosen constructor is explicit in copy-initialization > > return { profile, recommendedIndexingType }; > > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > /usr/bin/../lib/gcc/x86_64-linux-gnu/5.4.0/../../../../include/c++/5.4.0/ > > tuple:612:19: note: constructor declared here > > constexpr tuple(_U1&& __a1, _U2&& __a2) > > ^ > > 1 error generated. > > > > > > Log: > > https://build.webkit.org/builders/GTK%20Linux%2064- > > bit%20Release%20Ubuntu%20LTS%20%28Build%29/builds/13359/steps/compile-webkit/ > > logs/stdio > > There probably just needs to be a WTFMove around profile and > recommendedIndexingType. That didn't worked.. I was only able to fix the build error by reverting changes of r233065 over UnlinkedCodeBlock.h I have submitted a patch for fixing this at bug 187051 .. if you can review it that will be great. |