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 >> InstObjParams > object. >> >> Gabe > > > _______________________________________________ > 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
