This is an archive of the discontinued LLVM Phabricator instance.

NeonEmitter: remove special 'a' modifier.
ClosedPublic

Authored by t.p.northover on Nov 1 2019, 7:55 AM.

Details

Reviewers
efriedma
Summary

This has been separated off from D69618 to reduce diff clutter, as it really stands on its own. The 'a' modifier is rarely used, but emits splatting code from NeonEmitter.cpp when it is. This functionality can be implemented directly in .td files though, at a net reduction in lines of code.

There are a few test changes, but they're just moving bitcasts around due to the slightly different input Clang sees. Nothing really changes.

Diff Detail

Event Timeline

t.p.northover created this revision.Nov 1 2019, 7:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 1 2019, 7:55 AM
Herald added a subscriber: mcrosier. · View Herald Transcript
This revision is now accepted and ready to land.Nov 1 2019, 12:54 PM
t.p.northover closed this revision.Nov 6 2019, 2:24 AM

Thanks:

To github.com:llvm/llvm-project.git

6c2a4f5ff93..59f063b89c5  master -> master