dmgreen added a comment. In D131547#3728842 <https://reviews.llvm.org/D131547#3728842>, @sdesmalen wrote:
> In D131547#3723083 <https://reviews.llvm.org/D131547#3723083>, @dmgreen wrote: > >> I think we usually try to do the same, if the intrinsics have been in >> released compilers. There is an example in >> https://reviews.llvm.org/D98487#change-tOTTgECYYAO5, hopefully these would >> be equally simple. > > We don't really have the intention in keeping compatibility for intrinsics > like these, since the vector.extract/insert intrinsics are the proper way to > insert/extract vectors and they have been around for quite some time now. > Also these intrinsics are quite specific to the initial SVE ACLE > implementation when we first upstreamed it, so it's not very likely that > anyone else is using them. I'm not sure what you mean by "keeping compatibility", they just need to be replaced in the autoupdate code. If there was a release of llvm which emitted the intrinsics, then the default assumption is that someone might have bitcode that uses them. Luckily it is really simple to add the autoupgrade code in most cases. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131547/new/ https://reviews.llvm.org/D131547 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
