dmgreen added a comment. Looks good as far as I can see.
================ Comment at: clang/include/clang/Basic/arm_mve.td:78 + def _gather_base: Intrinsic< + Vector, (args VecOf<Unsigned<Scalar>>:$addr, imm_mem7bit<size>:$offset), + (IRInt<"vldr_gather_base", [Vector, VecOf<Unsigned<Scalar>>]> $addr, $offset)>; ---------------- Is it worth giving a name to VecOf<Unsigned<Scalar>>? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69791/new/ https://reviews.llvm.org/D69791 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits