Hi George,

Any chance of r27882 being reverted?

As I told the Fedora guys when that patch originally surfaced[1],
I'm not overly fond of
- copying source files around as part of the configure step
- having separate source files for ARMv6 and ARMv7, when those differences
  should be easily separated through macros (and would be reusable for 32-bit
  ARMv8).

Also, I might have mentioned that bit only on a separate thread on the Fedora 
list, but the ARMv4 support isn't actually correct (the ASM uses ARMv5-only 
operations).

My alternate solution, the basic idea of which I posted over there [2] was to 
separate ARMv5 and earlier from ARM. Effectively separating the atomics 
implementation at the boundary where The ARM architecture got 
load-linked/store-conditional, rather than having a separate source file for 
every architecture version.

[1] https://lists.fedoraproject.org/pipermail/arm/2012-November/004434.html
[2] https://lists.fedoraproject.org/pipermail/arm/2012-November/004460.html

Best Regards,

Leif

-- IMPORTANT NOTICE: The contents of this email and any attachments are 
confidential and may also be privileged. If you are not the intended recipient, 
please notify the sender immediately and do not disclose the contents to any 
other person, use it for any purpose, or store or copy the information in any 
medium.  Thank you.


Reply via email to