public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* CRIS atomics revisited 0/4: summary
@ 2012-07-16  3:40 Hans-Peter Nilsson
  0 siblings, 0 replies; only message in thread
From: Hans-Peter Nilsson @ 2012-07-16  3:40 UTC (permalink / raw)
  To: gcc-patches

These were spotted while debugging usage of atomics within
glibc.  The kind of changes are microoptimizations,
nanooptimizations, a buglet and a major issue.  Micro: the
load-store-conditional sequence for compare-and-swap I
originally committed was an earlier version improved later.
Nanooptimizations: choosing better-fitting operands for the
atomic operator insn.  Buglet: a post-increment could have
sneaked into the (non-atomic) arithmetic operator operand;
better make it nonmemory_operand altogether.  I also threw in
use of the now generic need_atomic_barrier_p, let's call that a
microoptimization.  The major issue, giving up on alignment of
atomic data by default, is last.  Tested together and some
separately, no regressions for cris-elf nor crisv32-elf.
Committed separately.

brgds, H-P

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2012-07-16  3:40 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-07-16  3:40 CRIS atomics revisited 0/4: summary Hans-Peter Nilsson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).