https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96991

--- Comment #4 from Tom de Vries <vries at gcc dot gnu.org> ---
(In reply to Jakub Jelinek from comment #3)
> Note, the testcase would need to require int128 effective target or even
> sync_int_128_runtime, ensure linking with -latomic on offloading targets
> that need it (unless we handle it automatically), but also as I said, it
> contains just 3 randomly chosen atomics.
> Would be nice to have more complete test coverage of the atomics, like
> having one test per basic atomic size (char, short, int, long long, int128)
> and in each test at least most of the __atomic* and __sync_* builtins that
> are out there.
> See e.g. gcc.dg/atomic*.c and gcc.dg/sync*.c.  Perhaps use macros to avoid
> too many repetitions.

For now, I'm going to commit without test-case.

I've tried for a while to make something that does not depend on libatomic, but
didn't manage.

And I didn't manage to xfail the resulting link failure due to no libatomic
support in a way that doesn't also xfail the ICE.

So, to be revisited once libatomic support for nvptx has been committed.

Reply via email to