Reviewers: Massi, fschneider, Message:
Also array_bounds_checks_elimination now seems to slight gain in kraken in my measurement (Nehalem 2.8G, both ia32 and x64). As desaturate doesn't regress and
guassian-blur remains a little faster. Zheng Liu [email protected] Description: BoundsCheck should be extended to support --array_bounds_checks_elimination TEST=mjsunit/array-bounds-check-removal.js BoundsCheck LIR implementation supposes op2=length belong to [0,int32_max], so "sub length, upper_offset" is not correct. Moved lower/upper offset handling to LIR because I feel it is simpler. Only fixed x86/x64 port. array_bounds_checks_elimination remains disabled, because I have no permission to check bug in http://code.google.com/p/v8/source/detail?r=11775 Zheng Liu [email protected] Please review this at http://codereview.chromium.org/10581014/ SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ Affected files: M src/arm/lithium-arm.cc M src/hydrogen-instructions.h M src/hydrogen-instructions.cc M src/ia32/lithium-codegen-ia32.cc M src/ia32/lithium-ia32.h M src/ia32/lithium-ia32.cc M src/mips/lithium-mips.cc M src/x64/lithium-codegen-x64.cc M src/x64/lithium-x64.h M src/x64/lithium-x64.cc M test/mjsunit/array-bounds-check-removal.js -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
