- 20 Aug, 2015 2 commits
-
-
mstarzinger authored
R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/1285183010 Cr-Commit-Position: refs/heads/master@{#30263}
-
sbonda authored
FlushICache should be NOP for Denver with part numbers 0x0, 0x1 and 0x2 only. Instruction cache needs to flushed for future versions of denver. Review URL: https://codereview.chromium.org/1287173004 Cr-Commit-Position: refs/heads/master@{#30262}
-
- 21 Jul, 2015 1 commit
-
-
dstence authored
Revisit of https://codereview.chromium.org/910333004. Use 4MB heap page size over the default of 1MB. This change provides an improvement of 1.86% on the composite octane benchmark score on PPC. This is 0.56% more than if --min_semi_space_size=4 was used to specify a 4MB heap page size. Additionally, two more tests required modification to account for configurable heap page size. R=svenpanne@chromium.org, hpayer@chromium.org, danno@chromium.org, mbrandy@us.ibm.com, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1221433022 Cr-Commit-Position: refs/heads/master@{#29775}
-
- 14 Jul, 2015 1 commit
-
-
balazs.kilvady authored
BUG= Review URL: https://codereview.chromium.org/1237753004 Cr-Commit-Position: refs/heads/master@{#29652}
-
- 13 Jul, 2015 1 commit
-
-
rmcilroy authored
Review URL: https://codereview.chromium.org/1221433021 Cr-Commit-Position: refs/heads/master@{#29604}
-
- 02 Jul, 2015 1 commit
-
-
dcheng authored
Clang warns if there are missing braces around a subobject initializer. The most common idiom that triggers this is: STRUCT s = {0}; if the first field of STRUCT is itself a struct. This can be more simply written as: STRUCT s = {}; which also prevents the warning from firing. BUG=chromium:505297 LOG=n Review URL: https://codereview.chromium.org/1219233003 Cr-Commit-Position: refs/heads/master@{#29437}
-
- 17 Jun, 2015 1 commit
-
-
svenpanne authored
The remaining uses need some non-mechanical work: * non-standard-layout type, probably due to mixed access control * extended field designators Review URL: https://codereview.chromium.org/1173343006 Cr-Commit-Position: refs/heads/master@{#29071}
-
- 12 Jun, 2015 3 commits
-
-
bmeurer authored
Up until now we used int32_t for NodeId, but that was not ideal because negative values are invalid for NodeId and we use it as an array index for example in the NodeMarker class, where C++ compilers on x64 have to generate code that does proper sign extension for the indices, which is completely unnecessary. R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/1178403004 Cr-Commit-Position: refs/heads/master@{#28997}
-
yangguo authored
Revert of MIPS64: Enable shorten-64-to-32 warning. (patchset #12 id:240001 of https://codereview.chromium.org/1133163005/) Reason for revert: Compile failure: https://chromegw.corp.google.com/i/client.v8/builders/V8%20Mac64%20-%20debug/builds/3070/steps/compile/logs/stdio Original issue's description: > MIPS64: Enable shorten-64-to-32 warning. > > Port cdc43bc5 > > Original commit message: > Enable clang's shorten-64-to-32 warning flag on ARM64, and fix the warnings > that arise. > > BUG= > > Committed: https://crrev.com/9af578a7c83b58a0ce25345998d9287cbf2030cb > Cr-Commit-Position: refs/heads/master@{#28990} TBR=danno@chromium.org,martyn.capewell@arm.com,paul.lind@imgtec.com,akos.palfi@imgtec.com,dusan.milosavljevic@imgtec.com,jkummerow@chromium.org,machenbach@chromium.org,svenpanne@chromium.org,balazs.kilvady@imgtec.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1182493007 Cr-Commit-Position: refs/heads/master@{#28991}
-
balazs.kilvady authored
Port cdc43bc5 Original commit message: Enable clang's shorten-64-to-32 warning flag on ARM64, and fix the warnings that arise. BUG= Review URL: https://codereview.chromium.org/1133163005 Cr-Commit-Position: refs/heads/master@{#28990}
-
- 01 Jun, 2015 1 commit
-
-
jacob.bramley authored
Use __ARM_ARCH_6ZK__ instead of __ARM_ARCH_6KZ__. The architecture is technically called ARMv6KZ, but GCC has always called it 6ZK. Clang defines __ARM_ARCH_6K__ for this target, and is unaffected. BUG=v8:3978 LOG=N Review URL: https://codereview.chromium.org/1145973004 Cr-Commit-Position: refs/heads/master@{#28709}
-
- 27 May, 2015 1 commit
-
-
machenbach authored
Revert of Use CLOCK_REALTIME_COARSE when available. (patchset #1 id:1 of https://codereview.chromium.org/1151283005/) Reason for revert: [Sheriff] This leads to several failures in chromium and blocks our roll: https://codereview.chromium.org/1154363002/ Bisect (https://codereview.chromium.org/1152553004/) points to this CL. Please add the failing chromium trybot on a reland of this CL. Original issue's description: > Use CLOCK_REALTIME_COARSE when available. > > On systems that have CLOCK_REALTIME_COARSE with good enough resolution, > we can avoid making a system call to get the current time; it's serviced > from the vDSO. > > This is v2 of the patch. v1 can be found at [0] but was reverted in [1] > because of Chromium sandbox restrictions. The necessary changes have > been applied upstream in [2]. > > [0] https://codereview.chromium.org/1125003002 > [1] https://codereview.chromium.org/1130083003 > [2] https://codereview.chromium.org/1133653002 > > BUG= > LOG=N > > Committed: https://crrev.com/28cea2b749f24ba33e6e0c8e343dd0d6258ee302 > Cr-Commit-Position: refs/heads/master@{#28639} TBR=jochen@chromium.org,bmeurer@chromium.org,ben@strongloop.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1154783003 Cr-Commit-Position: refs/heads/master@{#28656}
-
- 26 May, 2015 1 commit
-
-
ben authored
On systems that have CLOCK_REALTIME_COARSE with good enough resolution, we can avoid making a system call to get the current time; it's serviced from the vDSO. This is v2 of the patch. v1 can be found at [0] but was reverted in [1] because of Chromium sandbox restrictions. The necessary changes have been applied upstream in [2]. [0] https://codereview.chromium.org/1125003002 [1] https://codereview.chromium.org/1130083003 [2] https://codereview.chromium.org/1133653002 BUG= LOG=N Review URL: https://codereview.chromium.org/1151283005 Cr-Commit-Position: refs/heads/master@{#28639}
-
- 19 May, 2015 1 commit
-
-
jkummerow authored
Porting Chromium's https://codereview.chromium.org/1142783002/ to fix a division-by-zero bug. BUG=chromium:488302 LOG=n Review URL: https://codereview.chromium.org/1135293003 Cr-Commit-Position: refs/heads/master@{#28475}
-
- 15 May, 2015 1 commit
-
-
martyn.capewell authored
Enable clang's shorten-64-to-32 warning flag on ARM64, and fix the warnings that arise. BUG= Review URL: https://codereview.chromium.org/1131573006 Cr-Commit-Position: refs/heads/master@{#28412}
-
- 07 May, 2015 2 commits
-
-
machenbach authored
Revert of Use CLOCK_REALTIME_COARSE when available. (patchset #3 id:40001 of https://codereview.chromium.org/1125003002/) Reason for revert: [Sheriff] This causes layout test crashes: http://build.chromium.org/p/client.v8/builders/V8-Blink%20Linux%2064%20%28dbg%29/builds/2735 You can add the trybot v8_linux_layout_dbg on reland. Original issue's description: > Use CLOCK_REALTIME_COARSE when available. > > On systems that have CLOCK_REALTIME_COARSE with good enough resolution, we can > avoid making a system call to get the current time; it's serviced from the vDSO. > > BUG= > LOG=N > > Committed: https://crrev.com/537ed7500c087786f28f51ff5222f1c2113776d3 > Cr-Commit-Position: refs/heads/master@{#28280} TBR=jochen@chromium.org,deanm@chromium.org,bmeurer@chromium.org,ben@strongloop.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1130083003 Cr-Commit-Position: refs/heads/master@{#28288}
-
ben authored
On systems that have CLOCK_REALTIME_COARSE with good enough resolution, we can avoid making a system call to get the current time; it's serviced from the vDSO. BUG= LOG=N Review URL: https://codereview.chromium.org/1125003002 Cr-Commit-Position: refs/heads/master@{#28280}
-
- 05 May, 2015 1 commit
-
-
bmeurer authored
These macros are not needed anymore, so there's no point in supporting them. Review URL: https://codereview.chromium.org/1123723003 Cr-Commit-Position: refs/heads/master@{#28214}
-
- 04 May, 2015 1 commit
-
-
alph authored
Tick event processor should not stay in a tight loop when there's nothing to do. It can go sleep until next sample event. LOG=N BUG=v8:3967 Review URL: https://codereview.chromium.org/1118533003 Cr-Commit-Position: refs/heads/master@{#28211}
-
- 30 Apr, 2015 1 commit
-
-
bmeurer authored
The method is not used anywhere, and it is a bad idea in general anyway. If you see a need to call YieldCPU, then you're code is probably in need of a redesign! R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/1116853002 Cr-Commit-Position: refs/heads/master@{#28147}
-
- 28 Apr, 2015 1 commit
-
-
bmeurer authored
Currently only the Win64 bots report this warnings, which adds quite some overhead to the development process. With this flag we also get compiler warnings about implicit 64bit to 32bit truncations when building with clang on Linux/x64 and Mac/x64. R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/1111733002 Cr-Commit-Position: refs/heads/master@{#28093}
-
- 20 Apr, 2015 2 commits
-
-
svenpanne authored
Review URL: https://codereview.chromium.org/1098863003 Cr-Commit-Position: refs/heads/master@{#27947}
-
Ross McIlroy authored
R=jochen@chromium.org Review URL: https://codereview.chromium.org/1088993003 Cr-Commit-Position: refs/heads/master@{#27937}
-
- 08 Apr, 2015 1 commit
-
-
jing.bao authored
BUG=v8:4015 LOG=n Review URL: https://codereview.chromium.org/1040603002 Cr-Commit-Position: refs/heads/master@{#27648}
-
- 07 Apr, 2015 1 commit
-
-
bmeurer authored
Currently we only generate addresses in the range 2GiB to 4GiB, because of a bug, when running in 64-bit mode. With this fix we now use the full range 2GiB to 4096GiB. BUG=v8:3997 LOG=n R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/1051303003 Cr-Commit-Position: refs/heads/master@{#27610}
-
- 02 Apr, 2015 1 commit
-
-
jochen authored
BUG=none R=danno@chromium.org LOG=n Review URL: https://codereview.chromium.org/960903003 Cr-Commit-Position: refs/heads/master@{#27579}
-
- 31 Mar, 2015 1 commit
-
-
rmcilroy authored
The libdl library is already included on target builds of Android and needs to be added to the build command line with a particular order to avoid undefined references in other libraries. Fix this by only explicitly including it in host builds and relying on the implicit inclusion on target builds. Also remove the librt hack which is not longer necessary due to the AOSP build bot having been removed. BUG=chromium:469973 LOG=Y Review URL: https://codereview.chromium.org/1036133005 Cr-Commit-Position: refs/heads/master@{#27535}
-
- 23 Mar, 2015 1 commit
-
-
johan authored
as of https://reviews.freebsd.org/D848. This makes V8 build on FreeBSD-current (and the forthcoming 11). BUG=none LOG=n Review URL: https://codereview.chromium.org/1025823003 Cr-Commit-Position: refs/heads/master@{#27348}
-
- 19 Mar, 2015 2 commits
-
-
jochen authored
With this patch, it'll look like this: $ out/x64.optdebug/d8 --expose-trigger-failure test/mjsunit/mjsunit.js test/mjsunit/verify-assert-false.js ==== C stack trace =============================== 1: V8_Fatal 2: v8::internal::FunctionCallbackArguments::Call(void (*)(v8::FunctionCallbackInfo<v8::Value> const&)) 3: 0x727ced 4: 0x72b6ba 5: 0x188c7f607f9b BUG=none R=svenpanne@chromium.org LOG=y Review URL: https://codereview.chromium.org/1018313003 Cr-Commit-Position: refs/heads/master@{#27318}
-
weiliang.lin authored
BUG= Review URL: https://codereview.chromium.org/998883002 Cr-Commit-Position: refs/heads/master@{#27315}
-
- 12 Mar, 2015 1 commit
-
-
johan authored
e8b9f2d7 introduced a different cast which broke building on Illumos. Revert to previous behavior for V8_OS_SOLARIS only. Found on SmartOS while building with gcc 4.9.0. edit1: adding jochen to reviewers since he was assigned through the issue tracker and danno since he seems to do a fair amount of cross-platform work edit2: removing BUG reference because I don't understand what LOG needs to contain (and it seems to link to chromium and not the v8 repo). Please edit commit message as appropriate. BUG=3935 LOG=n Review URL: https://codereview.chromium.org/990063002 Cr-Commit-Position: refs/heads/master@{#27163}
-
- 04 Mar, 2015 1 commit
-
-
thakis authored
Shouldn't make a difference in practice, but it's a bit more readable and it gets the case of a 0 shift correct without undefined behavior. BUG=463436 LOG=N Review URL: https://codereview.chromium.org/975283002 Cr-Commit-Position: refs/heads/master@{#26975}
-
- 03 Mar, 2015 1 commit
-
-
sejunho authored
Problem: Excuting with flags as "--prof --logfile-per-isolate --logfile=/path/to/filename" expected file name: /path/to/isolate-<isolate id>-filename current result: isolate-<isolate id>-/path/to/filename This patch makes the file name we expected. Review URL: https://codereview.chromium.org/960813004 Cr-Commit-Position: refs/heads/master@{#26955}
-
- 19 Feb, 2015 1 commit
-
-
tfarina authored
BUG=None R=bmeurer@chromium.org LOG=N Review URL: https://codereview.chromium.org/933673002 Cr-Commit-Position: refs/heads/master@{#26733}
-
- 30 Jan, 2015 4 commits
-
-
bmeurer authored
R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/877753007 Cr-Commit-Position: refs/heads/master@{#26346}
-
michael_dawson authored
Contribution of PowerPC port (continuation of 422063005 and 817143002). This patch covers the key changes needed to the common files needed to support AIX. Subsequent patches will cover: - changes to update the ppc directories so they are current with the changes in the rest of the project. - remaining AIX changes not resolved by 4.8 compiler - individual optimizations for PPC This is based off of the GitHub repository https://github.com/andrewlow/v8ppc R=danno@chromium.org, svenpanne@chromium.org BUG= Review URL: https://codereview.chromium.org/866843003 Cr-Commit-Position: refs/heads/master@{#26343}
-
Benedikt Meurer authored
This reverts commit 6a4c0a3b and commit 0deaa4b6 for breaking GCC bots. TBR=svenpanne@chromium.org Review URL: https://codereview.chromium.org/893533003 Cr-Commit-Position: refs/heads/master@{#26342}
-
bmeurer authored
R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/888613002 Cr-Commit-Position: refs/heads/master@{#26340}
-
- 27 Jan, 2015 1 commit
-
-
bmeurer authored
BUG=chromium:452033, v8:3846 LOG=y R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/878063002 Cr-Commit-Position: refs/heads/master@{#26288}
-
- 23 Jan, 2015 1 commit
-
-
Ben Noordhuis authored
BUG=v8:3844 LOG=y R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/867713003 Patch from Ben Noordhuis <info@bnoordhuis.nl>. Cr-Commit-Position: refs/heads/master@{#26234}
-