Great. Would you like your name on the patch?
Thanks, Ali On 08.03.2012 23:17, Brian Grayson wrote: > Ali, I ran your patch through a few billion instructions, and the results > look identical to what I got with my more-complicated patch, so it looks > fine! > > Brian Grayson > > -----Original Message----- > From: [email protected] [mailto:[email protected]] On Behalf > Of Gabriel Michael Black > Sent: Thursday, March 08, 2012 8:39 PM > To: [email protected] > Subject: Re: [gem5-dev] ARM cbz/cbnz bug (and fix) > > Ok, sounds good. > > Gabe > > Quoting Ali Saidi <[email protected]>: > >> Yea, except I don't think that will work in this case, because IsUncondControl would get set anyway. Additionally, since the BranchRegImm is only used for the instructions in question, it can just be changed. Ali On 08.03.2012 18:23, Gabe Black wrote: >> >>> You can >> specify extra instruction flags using the ISA parser without >> >>> having to >> have a specialized constructor. It's an input to the Gabe _______________________________________________ gem5-dev mailing list > > _______________________________________________ > gem5-dev mailing list > [email protected] > http://m5sim.org/mailman/listinfo/gem5-dev > > _______________________________________________ > gem5-dev mailing list > [email protected] > http://m5sim.org/mailman/listinfo/gem5-dev _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
