Page MenuHomePhabricator
Differential D3496

ARM: remove @llvm.arm.sevl intrinsic
ClosedPublic
Actions

Authored by abdulras on Apr 24 2014, 4:09 PM.

Details

Reviewers
t.p.northover
abdulras
Summary

With the introduction of the @llvm.arm.hint intrinsic, this is no longer needed.
The users of this intrinsic can spell this as '@llvm.arm.hint(i32 5)'.

Depends on http://reviews.llvm.org/D3495.

Diff Detail

Event Timeline

abdulras updated this revision to Diff 8822.Apr 24 2014, 4:09 PM
abdulras retitled this revision from to ARM: remove @llvm.arm.sevl intrinsic.
abdulras updated this object.
abdulras edited the test plan for this revision. (Show Details)
abdulras added a reviewer: t.p.northover.
abdulras set the repository for this revision to rL LLVM.
abdulras added a subscriber: Unknown Object (MLST).
Herald added a subscriber: aemerson. · View Herald TranscriptApr 24 2014, 4:09 PM
t.p.northover edited edge metadata.Apr 25 2014, 3:51 AM

Hi Saleem,

This looks good to me (obviously, once @llvm.arm.hint is committed).

Cheers.

Tim.

abdulras accepted this revision.Apr 25 2014, 10:58 AM
abdulras added a reviewer: abdulras.

Committed as SVN r207246.

This revision is now accepted and ready to land.Apr 25 2014, 10:58 AM
abdulras closed this revision.Apr 25 2014, 10:58 AM

Revision Contents
Changeset List

PathSize
include/
llvm/
IR/
1 line
lib/
Target/
ARM/
2 lines
2 lines
test/
CodeGen/
ARM/
4 lines

Diff 8822

View Options

include/llvm/IR/IntrinsicsARM.td

Loading...
View Options

lib/Target/ARM/ARMInstrInfo.td

Loading...
View Options

lib/Target/ARM/ARMInstrThumb.td

Loading...
View Options

test/CodeGen/ARM/intrinsics-v8.ll

Loading...