This is an archive of the discontinued LLVM Phabricator instance.

[BOLT][test] Fix X86 tests
ClosedPublic

Authored by yota9 on Apr 5 2022, 8:09 AM.

Details

Summary

Be aware that the unreachable.test is marked as XFAIL due to passed cxxflags

Diff Detail

Event Timeline

yota9 created this revision.Apr 5 2022, 8:09 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 5 2022, 8:09 AM
Herald added a subscriber: ayermolo. · View Herald Transcript
yota9 requested review of this revision.Apr 5 2022, 8:09 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 5 2022, 8:09 AM
maksfb accepted this revision.Apr 5 2022, 8:22 AM

Thanks! Could you please check that premerge check failures are unrelated?

This revision is now accepted and ready to land.Apr 5 2022, 8:22 AM
yota9 updated this revision to Diff 420528.Apr 5 2022, 8:41 AM

Remove unreachable test changes

yota9 updated this revision to Diff 420537.Apr 5 2022, 9:05 AM

Update stubs

yota9 updated this revision to Diff 420541.EditedApr 5 2022, 9:16 AM

@maksfb Yes, you're right. The problem is coming from cxxflags passed to the test now, I'm marking it as XFAIL: * now (see my comments in discord)

yota9 edited the summary of this revision. (Show Details)Apr 5 2022, 9:59 AM
Amir added a comment.Apr 5 2022, 9:13 PM

Can you please add the following to unreachable.s? This would fix the test and make XFAIL: * unneeded:

diff --git a/bolt/test/X86/Inputs/unreachable.s b/bolt/test/X86/Inputs/unreachable.s
index b8dcf4c05557..39f4be971c87 100644
--- a/bolt/test/X86/Inputs/unreachable.s
+++ b/bolt/test/X86/Inputs/unreachable.s
@@ -82,4 +82,5 @@ foo:
        .globl  main, function
 main:
        .cfi_startproc
+  ud2
        .cfi_endproc
yota9 updated this revision to Diff 420844.Apr 6 2022, 7:35 AM

Fix unreachable.s

yota9 updated this revision to Diff 420899.Apr 6 2022, 8:50 AM

clang-format

yota9 added a comment.Apr 6 2022, 10:15 AM

@Amir Thank you very much, it works now, submitting the patch now :)

This revision was automatically updated to reflect the committed changes.