Bug 78237
Summary: | [Qt][ARM] REGRESSION(r107126): It broke zillion tests | ||
---|---|---|---|
Product: | WebKit | Reporter: | Csaba Osztrogonác <ossy> |
Component: | JavaScriptCore | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED FIXED | ||
Severity: | Critical | CC: | barraclough, loki, oliver, ossy, pvarga, rgabor, zherczeg |
Priority: | P1 | Keywords: | Qt, QtTriaged |
Version: | 528+ (Nightly build) | ||
Hardware: | All | ||
OS: | All | ||
Bug Depends on: | |||
Bug Blocks: | 78145, 79666 |
Csaba Osztrogonác
See http://build.webkit.sed.hu/builders/ARMv5%20Linux%20Qt%20Release%20%28Test%29/builds/5708 for details
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Oliver Hunt
What endianness does the Qt ARM bot use?
Gabor Rapcsanyi
We are using little endian (armel).
(In reply to comment #1)
> What endianness does the Qt ARM bot use?
Csaba Osztrogonác
Now tests work with a newer GCC, but it is paper overing the problem. Zoltán commented the original bug, why this change is serious, it is a time-bomb.
Oliver Hunt
(In reply to comment #3)
> Now tests work with a newer GCC, but it is paper overing the problem. Zoltán commented the original bug, why this change is serious, it is a time-bomb.
Oh right, see my reply to Zoltan -- it's a bug in the ARM assembler if you're planting different sized instructions in regions we expect to be repatched.
Csaba Osztrogonác
It seems it works now.