On Thu, Jun 16, 2011 at 4:54 PM, Fang, Changpeng <changpeng.f...@amd.com> wrote: > Hi, > > I modify the patch to disable unaligned load splitting only for bdver1 at > this moment. > Unaligned load splitting degrades CFP2006 by 1.3% in geomean for both > -mtune=bdver1 and > -mtune=generic on Bulldozer. However, we agree with H.J's suggestion to > determine > the optimal optimization sets for modern cpus. > > Is is OK to commit the attached patch? > > Thanks,
Why not just move AVX256_SPLIT_UNALIGNED_STORE and AVX256_SPLIT_UNALIGNED_LOAD to ix86_tune_indices? H.J. > Changpeng > > > >>> So, is it OK to commit this patch to trunk, and H.J's original patch + this >>> to 4.6 branch? >>> >>> > >>I have no problems on -mtune=Bulldozer. But I object -mtune=generic >>change and did suggest a different approach for -mtune=generic. > > > . > > -- H.J.