This is an archive of the discontinued LLVM Phabricator instance.

Add cdp intrinsic tests
ClosedPublic

Authored by rs on May 18 2016, 3:43 PM.

Details

Summary

I've attached a patch which adds tests to make sure the cdp intrinsics fault when trying to pass a variable as an argument. I've also renamed the intrinsics.ll to intrinsics-coprocessor.ll as the only intrinsics that file tested were coprocessor intrinsics and made the checks for specific.

Diff Detail

Event Timeline

rs updated this revision to Diff 57690.May 18 2016, 3:43 PM
rs retitled this revision from to Add cdp intrinsic tests.
rs updated this object.
rs added a reviewer: rengolin.
rs added a subscriber: llvm-commits.
rs updated this revision to Diff 57694.May 18 2016, 3:47 PM
rengolin accepted this revision.May 18 2016, 5:16 PM
rengolin edited edge metadata.

LGTM. Thanks!

This revision is now accepted and ready to land.May 18 2016, 5:16 PM
This revision was automatically updated to reflect the committed changes.
test/CodeGen/ARM/intrinsics-coprocessor.ll