This is an archive of the discontinued LLVM Phabricator instance.

Fix the Thumb test for vfloat intrinsics
ClosedPublic

Authored by pbarrio on Sep 9 2016, 8:05 AM.

Details

Summary

This test was not testing the intrinsics. A function like this:

define %v4f32 @test_v4f32.floor(%v4f32 %a){
...

%1 = call %v4f32 @llvm.floor.v4f32(%v4f32 %a)

...
}

is transformed into the following assembly:

_test_v4f32.floor: @ @test_v4f32.floor
...

bl _floorf

...

In each function tested, there are two CHECK: one that checked
for the label and another one for the intrinsic that should be used
inside the function (in our case, "floor"). However, although the
first CHECK was matching the label, the second was not matching the
intrinsic, but the second "floor" in the same line as the label.

This is fixed by making the first CHECK match the entire line.

Diff Detail

Repository
rL LLVM

Event Timeline

pbarrio updated this revision to Diff 70838.Sep 9 2016, 8:05 AM
pbarrio retitled this revision from to Fix the Thumb test for vfloat intrinsics.
pbarrio updated this object.
pbarrio added a reviewer: jmolloy.
pbarrio added a subscriber: llvm-commits.

Hello James,

Can you accept this patch again, now as a standalone change? I have added and fixed the missing test, as per Renato's suggestion in D24337.

Cheers,
Pablo

rengolin accepted this revision.Sep 9 2016, 8:37 AM
rengolin added a reviewer: rengolin.

LGTM, thanks!

This revision is now accepted and ready to land.Sep 9 2016, 8:37 AM
This revision was automatically updated to reflect the committed changes.