This is an archive of the discontinued LLVM Phabricator instance.

Define TrapInst for ppc64
ClosedPublic

Authored by espindola on Apr 2 2018, 1:30 PM.

Details

Reviewers
sfertile
ruiu
Summary

This is nice for testing since it is the first TrapInst whose bytes are not all the same.

Diff Detail

Event Timeline

espindola created this revision.Apr 2 2018, 1:30 PM
ruiu added inline comments.Apr 2 2018, 1:47 PM
test/ELF/fill-trap.s
1 ↗(On Diff #140682)

Don't you need to enable both x86 and ppc to run this test now? Maybe it is better to split this for each arch.

Use a new test file for ppc64.

ruiu accepted this revision.Apr 2 2018, 2:07 PM

LGTM

This revision is now accepted and ready to land.Apr 2 2018, 2:07 PM