JonChesterfield added a comment. In D82087#3803464 <https://reviews.llvm.org/D82087#3803464>, @arsenm wrote:
> In D82087#3797883 <https://reviews.llvm.org/D82087#3797883>, @jdoerfert wrote: > >> Can we land this? I'd like to use the new intrinsics as I don't understand >> the old ones. > > What do you think about using the two separate builtins, vs. one magic > builtin that auto-changes the wavesize? The magic one would also change its return type, or always be i64 with high bits (zext or maybe sext or maybe copy of low), so less magic seems clearer CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82087/new/ https://reviews.llvm.org/D82087 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits