- 05 May, 2011 9 commits
-
-
dslomov@chromium.org authored
BUG= TEST= Committed: http://code.google.com/p/v8/source/detail?r=7734 Committed: http://code.google.com/p/v8/source/detail?r=7784 Review URL: http://codereview.chromium.org/6788023 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7797 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
http://codereview.chromium.org/6901141cira@chromium.org authored
Changes from previus revision: - Made my own strncpy in I18NUtils class (we can't use OS::SNPrintF nor snprintf). - Fixed a crashing bug related to ICU call in LanguageMatcher::BCP47ToICUFormat. TEST=Visit i18n.kaziprst.org/locale.html Review URL: http://codereview.chromium.org/6928017 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7796 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
vegorov@chromium.org authored
Review URL: http://codereview.chromium.org/6932010 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7795 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
vitalyr@chromium.org authored
In my previous change I mixed up "compare" and "equals". This made us miss the fast length check before comparing strings for equality. Now we have a separate helper for "equals". It shares some of the inner loop details with "compare". I'll see if this can be cleaned up without making it unreadable. Review URL: http://codereview.chromium.org/6928020 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7794 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ager@chromium.org authored
R=fschneider@chromium.org BUG= TEST= Review URL: http://codereview.chromium.org/6930005 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7792 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
Whenever >>> produces a non-smi result, the binary op stub would return a wrong result in case of a failed new-space allocation. This also fixes the (flaky) sparse-array-reverse test which was failing sometimes because of this. BUG=v8:1359 TEST=mjsunit/sparse-array-reverse.js Review URL: http://codereview.chromium.org/6927013 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7790 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ager@chromium.org authored
Don't copy 64 bits to a 32 bit value. R=karlklose@chromium.org,sgjesse@chromium.org BUG= TEST= Review URL: http://codereview.chromium.org/6933009 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7788 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ricow@chromium.org authored
Fixes mozilla failures Review URL: http://codereview.chromium.org/6935020 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7787 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ricow@chromium.org authored
The only difference to revision 7763 is the implementation in the builtins file for arm and x64, plus a move of Array.prototype.toString and Array.prototype.toLocaleString from should throw on null or undefined to the non generic test cases in the function-call test (due to us not currently supporting generic cases with these to functions) Review URL: http://codereview.chromium.org/6928007 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7786 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 04 May, 2011 13 commits
-
-
dslomov@chromium.org authored
This reverts commit 1aa40d0c278e195e9e69dcf8b29a212aa1817c0b. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7785 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
dslomov@chromium.org authored
BUG= TEST= Committed: http://code.google.com/p/v8/source/detail?r=7734 Review URL: http://codereview.chromium.org/6788023 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7784 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
vitalyr@chromium.org authored
BUG=1340 Review URL: http://codereview.chromium.org/6937001 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7783 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
whesse@chromium.org authored
Tiny refactoring - change compilation phase parameter for CopyForInlining from a boolean to an enum. BUG= TEST= Review URL: http://codereview.chromium.org/6913021 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7782 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ager@chromium.org authored
Patch by Peter Varga. BUG=v8:1336 TEST=cctest/test-api/CallAsFunction Review URL: http://codereview.chromium.org/6883045 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7781 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
Review URL: http://codereview.chromium.org/6933004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7778 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
sgjesse@chromium.org authored
This broke the Chromium Windows shared build. BUG=none TEST=Windows shared build works when pushed to Chromium Review URL: http://codereview.chromium.org/6932007 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7777 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
sgjesse@chromium.org authored
R=fschneider@chromium.org BUG=none TEST=Windows shared build works when pushed to Chromium Review URL: http://codereview.chromium.org//6932006 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7776 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ager@chromium.org authored
Tests are failing in debug mode. This also reapplies the change "Make Date and RegExp inherit from Object in the API" that I accidentally reverted. R=sgjesse@chromium.org BUG= TEST= Review URL: http://codereview.chromium.org/6927009 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7775 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ager@chromium.org authored
Tests are failing in debug mode. R=sgjesse@chromium.org BUG= TEST= Review URL: http://codereview.chromium.org/6930004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7774 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ager@chromium.org authored
Patch by Peter Varga. BUG=v8:1336 TEST=cctest/test-api/CallAsFunction Review URL: http://codereview.chromium.org/6883045 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7773 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ager@chromium.org authored
They are objects and the API inheritance hierarchy should reflect that. R=sgjesse@chromium.org BUG= TEST= Review URL: http://codereview.chromium.org/6929008 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7772 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
sgjesse@chromium.org authored
Added the GYP variables: v8_can_use_unaligned_accesses v8_use_arm_eabi_hardfloat Refer to the comments in v8.gyp on their uses. R=karlklose@chromium.org BUG=none TEST=none Review URL: http://codereview.chromium.org//6910025 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7769 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 03 May, 2011 14 commits
-
-
cira@chromium.org authored
Change v8Locale to match proposal - constructor is different (takes settings object, not a plain string) and I've added derive method to it. Added comments to i18n.js methods and properties, and util functions to check settings and locale validity. Added LanguageMatcher class until ICU gets C implementation (in progress, but late for our current deadline). I added TODO to remove LanguageMatcher code. TEST=Visit http://i18n.kaziprst.org/locale.html. Review URL: http://codereview.chromium.org/6901141 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7768 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
Review URL: http://codereview.chromium.org/6909026 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7767 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ricow@chromium.org authored
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7766 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
karlklose@chromium.org authored
Replace long running loops by OptimizeFunctionOnNextCall in some tests that are often timing out on ARM. Review URL: http://codereview.chromium.org/6910022 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7765 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ricow@chromium.org authored
Review URL: http://codereview.chromium.org/6913024 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7764 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ricow@chromium.org authored
Don't exchange null and undefined with the global object in function.prototype.{call, apply} for natives. This makes us compatible with firefox in throwing an exception when call is invoked on a builtin with null as the this argument. Review URL: http://codereview.chromium.org/6902104 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7763 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
lrn@chromium.org authored
Fix issue 1356 BUG=v8:1356 TEST=mjsunit/double-equals Review URL: http://codereview.chromium.org/6912021 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7761 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ager@chromium.org authored
Encoding is one byte shorter. R=lrn@chromium.org BUG= TEST= Review URL: http://codereview.chromium.org/6910021 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7760 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
generalization of the latter. This makes CallWrapper architecture-independant, so it can be pulled up into assembler.h, nuking 3 copy-n-paste classes. Only a small improvement, but nevertheless... Review URL: http://codereview.chromium.org/6893156 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7758 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
Until now we conservatively chose a double representation if at least one use occurs in a double operation. This causes performance degradation in many cases where there are mixes uses (integer and double) e.g.: for (int i = 0; i < 10; i++) { var t = i / 3.5; a[i] = t; } where the use in i/3 requires a double, where as the keyed store requires i as an integer. For these cases we want to have i as an integer and convert it only before the double division. In order to avoid unconditional deoptimization in some rare cases, we check phis if there is any conversion that will always fail when converting a heap-number constant to int32. Review URL: http://codereview.chromium.org/6905166 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7757 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
kmillikin@chromium.org authored
Try to make sure that accessors.h, data-flow.h, list-inl.h, and scopeinfo.h are included only where needed, but without introducing implicit dependencies. Review URL: http://codereview.chromium.org/6903175 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7756 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
lrn@chromium.org authored
BUG=v8:955 TEST=mjsunit/regress/regress-955 Review URL: http://codereview.chromium.org/6903171 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7755 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
sgjesse@chromium.org authored
Always use CpuFeaturesImpliedByCompiler() when selecting CPU features. This checks both for CAN_USE_ARMV7_INSTRUCTIONS and CAN_USE_VFP_INSTRUCTIONS and for GCC preprocessor symbols. This will support using the CAN_USE_XXX for a simulator build used for generating a snapshot followed by a crosscompile using -march= and -mfpu= for selecting the (minimal) target device CPU features. The snapshot will use instructions based on the CAN_USE_XXX whereas the target will at least use features based on both CAN_USE_XXX and -march= and -mfpu=, but will try runtime CPU feature detection a well looking for somethis better. Remove the compiler based CPU feature detection from the OS::CpuFeaturesImpliedByPlatform() as it did not belong there. Also was already in the CpuFeaturesImpliedByCompiler(). Add the variable 'v8_can_use_vfp_instructions' to the GYP file which can be used to turn on CAN_USE_VFP_INSTRUCTIONS when building V8. I did not add any -mfpu= cflags for this, as there are several options here (e.g. vfp and neon). R=erik.corry@gmail.com, karlklose@chromium.org BUG=none TEST=none Review URL: http://codereview.chromium.org//6904164 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7754 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ager@chromium.org authored
R=sgjesse@chromium.org BUG=v8:1258 TEST=test-api/AllowCodeGenFromStrings Review URL: http://codereview.chromium.org/6905167 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7753 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 02 May, 2011 4 commits
-
-
ager@chromium.org authored
TBR=sgjesse@chromium.org Review URL: http://codereview.chromium.org/6905168 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7752 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ager@chromium.org authored
Review URL: http://codereview.chromium.org/6902128 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7751 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ager@chromium.org authored
R=sgjesse@chromium.org BUG= TEST= Review URL: http://codereview.chromium.org/6903172 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7748 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
sgjesse@chromium.org authored
R=ager@chromium.org Review URL: http://codereview.chromium.org/6902200 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7747 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-