1480706 - [clang linux32] TEST-UNEXPECTED-FAIL | LulIntegration.unwind_consistency | Value of: nTestsPassed == nTests

Status: RESOLVED FIXED

(Core :: Gecko Profiler, defect)

Description

[Mike Hommey [:glandium]]

Attachment: LUL log

Switching linux32 builds to clang fails gtests with:

TEST-UNEXPECTED-FAIL | LulIntegration.unwind_consistency | Value of: nTestsPassed == nTests

There is sadly not more in the output because of:
https://dxr.mozilla.org/mozilla-central/source/tools/profiler/tests/gtest/LulTest.cpp#12-15

This patch is enough to get going to reproduce the issue on try:
https://hg.mozilla.org/try/rev/bda9def26ee3d0808a87b5d0b9bdd035f3bd77b1

That builds Firefox with clang 6.0.1.

Attached is the LUL log I got on try after enabling DEBUG_LUL_TEST.

Comment 1

[Mike Hommey [:glandium]]

Note that building with clang requires apply the patch from bug 1480401.

Comment 2

[Mike Hommey [:glandium]]

Attachment: Enable Frame-pointer chasing on x86

Comment 3

[Julian Seward [:jseward]]

Comment on attachment 8998133 [details] [diff] [review]
Enable Frame-pointer chasing on x86

Note that that will only enable it for desktop x86(32)-linux.
If you want to also enable it for x86(32)-android, you'll need 
to throw in GP_PLAT_x86_android too.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/946dc9d537c0
Enable Frame-pointer chasing on x86. r=sewardj

Comment 5

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/946dc9d537c0