wenju-he wrote: > > > I think __clc_atomic_inc/dec should be deleted, or renamed. inc is a > > > different op (i.e., atomicrmw uinc_wrap) > > > > > > can we add new __scoped_atomic_uinc_wrap/udec_wrap clang builtin and use it > > for __clc_atomic_inc/dec implementation? our downstream implementation of > > __clc_atomic_inc does use uinc_wrap in atomicrmw. > > Yes, should do that
changed to use __scoped_atomic_uinc_wrap for __clc_atomic_inc and __scoped_atomic_udec_wrap for __clc_atomic_dec https://github.com/llvm/llvm-project/pull/168327 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
