[PATCH] D122189: [Clang][NeonEmitter] emit ret decl first for -Wdeclaration-after-statement

[Nick Desaulniers via Phabricator via cfe-commits]

nickdesaulniers added a comment.

In D122189#3399836 <https://reviews.llvm.org/D122189#3399836>, @DavidSpickett wrote:

> Can you post a bit of the header diff before/after? I think I know what it looks like but just to be sure.

Before:
#define __noswap_splat_lane_f16(__p0,__p1) __extension__ ({ float16x4_t __s0 = __p0; float16x4_t __ret; __ret = (float16x4_t) __builtin_neon_splat_lane_v((int8x8_t)__s0, __p1, 8); __ret; })
After:
#define __noswap_splat_lane_f16(__p0,__p1) __extension__ ({ float16x4_t __ret; float16x4_t __s0 = __p0; __ret = (float16x4_t) __builtin_neon_splat_lane_v((int8x8_t)__s0, __p1, 8); __ret; })

Before:
#define vld1_lane_f32(__p0,__p1,__p2) __extension__ ({ float32x2_t __s1 = __p1; float32x2_t __rev1; __rev1 = __builtin_shufflevector(__s1, __s1, 1, 0); float32x2_t __ret; __ret = (float32x2_t) __builtin_neon_vld1_lane_v(__p0, (int8x8_t)__rev1, __p2, 9); __ret = __builtin_shufflevector(__ret, __ret, 1, 0); __ret; })
After:
#define vld1_lane_f32(__p0,__p1,__p2) __extension__ ({ float32x2_t __ret; float32x2_t __s1 = __p1; float32x2_t __rev1; __rev1 = __builtin_shufflevector(__s1, __s1, 1, 0); __ret = (float32x2_t) __builtin_neon_vld1_lane_v(__p0, (int8x8_t)__rev1, __p2, 9); __ret = __builtin_shufflevector(__ret, __ret, 1, 0); __ret; })


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122189/new/

https://reviews.llvm.org/D122189