Hi, I've removed all the typos and formatting problems from the LLVM patch the attached the new one.
Thanks, Silviu > -----Original Message----- > From: Silviu Baranga [mailto:[email protected]] > Sent: 05 September 2012 16:49 > To: 'Anton Korobeynikov' > Cc: [email protected]; [email protected] > Subject: RE: [llvm-commits] [PATCH] Add the Cortex-A15 target to Clang > and LLVM > > Hi Anton, > > The next patch wouldn't actually remove any fixme statements. > It would remove a feature that we have determined to not be > profitable on A15. > > I will fix the typos and formatting problems and send a new > patch (I don't know how they get through). > > Thanks, > Silviu > > > -----Original Message----- > > From: Anton Korobeynikov [mailto:[email protected]] > > Sent: 05 September 2012 13:48 > > To: Silviu Baranga > > Cc: [email protected]; [email protected] > > Subject: Re: [llvm-commits] [PATCH] Add the Cortex-A15 target to > Clang > > and LLVM > > > > Hello Silviu, > > > > > keeps code generation for A15 identical to code generation for A9. > > The > > > patches > > > basically enable tuning code generation for A15. Once the patches > are > > > accepted, we have 1 more patch in the pipeline that starts to tune > > LLVM > > > code generation for A15. > > Will you please post this patch as well? It might be possible that it > > will make sense to get rid of some FIXMEs in the present patch > > immediately. > > > > You have some typos in the patch ("detemined") and also formatting > > problems, e.g. you have stuff like > > > > if (foo > > || bar) > > > > Thanks! > > > > -- > > With best regards, Anton Korobeynikov > > Faculty of Mathematics and Mechanics, Saint Petersburg State > University
llvm_A15.diff
Description: Binary data
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
