Hi, James, I could add arm1026ejs to ARMTargetInfo::getCPUDefineSuffix, but then it fails anyway because arm1026ejs is not in ARM.td.
I've added a test. On Tue, Apr 24, 2012 at 6:29 PM, James Molloy <[email protected]> wrote: > Hi Evgeniy, > > The patch itself appears fine but you're missing a testcase. Could you please > add one? The driver test suite is a bit lacking so for any fixes it would be > really nice to know if we regress. > > Also you mention ARMTargetInfo::getCPUDefineSuffix, but you didn't submit a > patch to fix that? > > Cheers, > > James > > -----Original Message----- > From: [email protected] > [mailto:[email protected]] On Behalf Of Evgeniy Stepanov > Sent: 24 April 2012 14:59 > To: [email protected] > Subject: [cfe-commits] [PATCH] Fix default cpu for -march=armv5e > > Hi, > > arm1026ejs is armv5tej, and, what's worse, the mapping of cpu name to > arch suffix in ARMTargetInfo::getCPUDefineSuffix does not know that it > exists. > > Please review. > > >
armv5e.patch
Description: Binary data
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
