| [eigen-commits] commit/eigen: rmlarsen: Revert PR #292. After further investigation, the memcpy->memmove change was only good for Haswell on older versions of glibc. Adding a switch for small sizes is perhaps useful for string copies, but also has an overhead for larger sizes, making it a poor trade-off for general memcpy. |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/eigen-commits Archives
]
- To: eigen-commits@xxxxxxxxxxxxxxxxxxx
- Subject: [eigen-commits] commit/eigen: rmlarsen: Revert PR #292. After further investigation, the memcpy->memmove change was only good for Haswell on older versions of glibc. Adding a switch for small sizes is perhaps useful for string copies, but also has an overhead for larger sizes, making it a poor trade-off for general memcpy.
- From: Bitbucket <commits-noreply@xxxxxxxxxxxxx>
- Date: Fri, 27 Jan 2017 17:25:02 -0000
- Dkim-signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=bitbucket.org; h=mime-version:content-type:content-transfer-encoding:subject:from:to; s=s1; bh=kMRaKAC5jQdt0OqShOVMtLPrKWs=; b=WEOEGOb92xfxgov7y0vgJOS U7cKj7ONI4QbJgSUlo/Fj84ZtapHT0DCrqqsFH5qY4xEfRh1ujIBhpA2qqM+K6kn 7DO+p79BucvH/lDQk4f2/HBTSvdtStwYSWtEcsDU02VoE4cPQdMNN/HGoALSU1ba RNCWBKIkbMy8w68qEWZM=
1 new commit in eigen:
https://bitbucket.org/eigen/eigen/commits/8fa870164ed7/
Changeset: 8fa870164ed7
User: rmlarsen
Date: 2017-01-26 20:46:06+00:00
Summary: Revert PR #292. After further investigation, the memcpy->memmove change was only good for Haswell on older versions of glibc. Adding a switch for small sizes is perhaps useful for string copies, but also has an overhead for larger sizes, making it a poor trade-off for general memcpy.
This PR also removes a couple of unnecessary semi-colons in Eigen/src/Core/AssignEvaluator.h that caused compiler warning everywhere.
Affected #: 5 files
Repository URL: https://bitbucket.org/eigen/eigen/
--
This is a commit notification from bitbucket.org. You are receiving
this because you have the service enabled, addressing the recipient of
this email.