Bug 157265

Summary:	New regexp JSC stress tests introduced in r200239 fail on ARMv7 Thumb2
Product:	WebKit	Reporter:	Csaba Osztrogonác <ossy>
Component:	JavaScriptCore	Assignee:	Yusuke Suzuki <ysuzuki>
Status:	RESOLVED DUPLICATE
Severity:	Normal	CC:	keith_miller, ossy, ysuzuki
Priority:	P2
Version:	Other
Hardware:	Unspecified
OS:	Unspecified
Bug Depends on:
Bug Blocks:	108645, 157124

Description Csaba Osztrogonác 2016-05-02 03:47:44 PDT

new tests introduced in https://trac.webkit.org/changeset/200239
fail on ARMv7 Thumb2 (JSCOnly port):
Source/JavaScriptCore/tests/stress/regexp-replace-in-other-realm-should-work.js
Source/JavaScriptCore/tests/stress/regexp-replace-should-work-with-objects-not-inheriting-object-prototype.js

before: (41 fails) https://build.webkit.org/builders/JSCOnly%20Linux%20ARMv7%20Thumb2%20Release/builds/423
after: (55 fails) https://build.webkit.org/builders/JSCOnly%20Linux%20ARMv7%20Thumb2%20Release/builds/424

new failing tests:
	stress/regexp-replace-in-other-realm-should-work.js.always-trigger-copy-phase
	stress/regexp-replace-in-other-realm-should-work.js.default
	stress/regexp-replace-in-other-realm-should-work.js.dfg-eager
	stress/regexp-replace-in-other-realm-should-work.js.dfg-eager-no-cjit-validate
	stress/regexp-replace-in-other-realm-should-work.js.dfg-maximal-flush-validate-no-cjit
	stress/regexp-replace-in-other-realm-should-work.js.no-cjit-validate-phases
	stress/regexp-replace-in-other-realm-should-work.js.no-llint
	stress/regexp-replace-should-work-with-objects-not-inheriting-object-prototype.js.always-trigger-copy-phase
	stress/regexp-replace-should-work-with-objects-not-inheriting-object-prototype.js.default
	stress/regexp-replace-should-work-with-objects-not-inheriting-object-prototype.js.dfg-eager
	stress/regexp-replace-should-work-with-objects-not-inheriting-object-prototype.js.dfg-eager-no-cjit-validate
	stress/regexp-replace-should-work-with-objects-not-inheriting-object-prototype.js.dfg-maximal-flush-validate-no-cjit
	stress/regexp-replace-should-work-with-objects-not-inheriting-object-prototype.js.no-cjit-validate-phases
	stress/regexp-replace-should-work-with-objects-not-inheriting-object-prototype.js.no-llint

Comment 1 Csaba Osztrogonác 2016-05-02 03:50:16 PDT

Running stress/regexp-replace-in-other-realm-should-work.js.always-trigger-copy-phase
stress/regexp-replace-in-other-realm-should-work.js.always-trigger-copy-phase: Exception: TypeError: RegExp.prototype.@@replace requires that |this| be an Object
stress/regexp-replace-in-other-realm-should-work.js.always-trigger-copy-phase: replace@[native code]
stress/regexp-replace-in-other-realm-should-work.js.always-trigger-copy-phase: global code@regexp-replace-in-other-realm-should-work.js:9:58
stress/regexp-replace-in-other-realm-should-work.js.always-trigger-copy-phase: ERROR: Unexpected exit code: 3
FAIL: stress/regexp-replace-in-other-realm-should-work.js.always-trigger-copy-phase

Comment 2 Csaba Osztrogonác 2016-05-02 06:49:32 PDT

A clean build fixed it: 
https://build.webkit.org/builders/JSCOnly%20Linux%20ARMv7%20Thumb2%20Release/builds/453

new bug report to fix the build system: bug157266

*** This bug has been marked as a duplicate of bug 157266 ***