Page MenuHomePhabricator

[NEON] Define half-precision vrnd intrinsics only when available
ClosedPublic

Authored by kosarev on Jul 16 2018, 7:31 AM.

Diff Detail

Repository
rC Clang

Event Timeline

kosarev created this revision.Jul 16 2018, 7:31 AM
SjoerdMeijer accepted this revision.Jul 23 2018, 1:41 AM

LGTM

include/clang/Basic/arm_neon.td
1420–1427

nit: is the indentation a bit off here?

This revision is now accepted and ready to land.Jul 23 2018, 1:41 AM
kosarev added inline comments.Jul 23 2018, 6:35 AM
include/clang/Basic/arm_neon.td
1420–1427

It's a nested let ArchGuard, so I guess we do want the indentation here?

SjoerdMeijer added inline comments.Jul 23 2018, 6:43 AM
include/clang/Basic/arm_neon.td
1420–1427

Yep, got confused, please ignore.

This revision was automatically updated to reflect the committed changes.