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 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
