I was thinking more like defining a base Event class with common elements and then base classes that do what the select statements would have specialized them to do. Then the enums become unnecessary and the problem goes away. I might give that a try one of these days, time permitting.
Gabe Korey Sewell wrote: > Thanks for that update Gabe. > > Interesting that 4.5.1 I guess is doing some types of bounds checking > on enums and since there is one enum list (RespoolEventType) that > starts its enumeration from the end of another enum list > (CPUEventType) then it kind of cries "not working!" at that point. > > The fix is probably just to merge the two lists and then to > annotate/comment that at a certain point they are more resource pool > specific then CPU specific. > > On Sun, Dec 5, 2010 at 2:53 AM, Gabriel Michael Black > <[email protected]> wrote: > >> The attached patch should fix it. In my opinion this fixes a symptom and the >> way events are organized in InOrder should probably be revisited to make it >> unnecessary, but it should at least get things building for you. >> >> Gabe >> >> Quoting "Sudhanshu(Duke)" <[email protected]>: >> >> >>> I am using gcc 4.5.1 on fedora 14-i686 >>> >>> >>> ************************************************************************** >>> >>> >>> What compiler are you using? It might be too old, in which case I'd >>> suggest upgrading if possible, or it might be too new in which case we >>> should try to make the code build correctly. >>> >>> Gabe >>> >>> Quoting "Sudhanshu(Duke)" <[email protected]>: >>> >>> >>> Hi >>> I was compiling m5sim and got following errors >>> >>> $ scons build/ALPHA_SE/m5.debug >>> . >>> . >>> . >>> [ CXX] ALPHA_SE/cpu/inorder/resource_pool.cc >>> cc1plus: warnings being treated as errors >>> build/ALPHA_SE/cpu/inorder/resource_pool.cc: In member function 'void >>> ResourcePool::scheduleEvent(InOrderCPU::CPUEventType, >>> ResourcePool::DynInstPtr, int, int, ThreadID)': >>> build/ALPHA_SE/cpu/inorder/resource_pool.cc:326:7: error: case value >>> '10' not in enumerated type 'InOrderCPU::CPUEventType' >>> build/ALPHA_SE/cpu/inorder/resource_pool.cc:358:7: error: case value >>> '11' not in enumerated type 'InOrderCPU::CPUEventType' >>> build/ALPHA_SE/cpu/inorder/resource_pool.cc: In member function >>> 'virtual void ResourcePool::ResPoolEvent::process()': >>> build/ALPHA_SE/cpu/inorder/resource_pool.cc:525:7: error: case value >>> '10' not in enumerated type 'InOrderCPU::CPUEventType' >>> build/ALPHA_SE/cpu/inorder/resource_pool.cc:533:7: error: case value >>> '11' not in enumerated type 'InOrderCPU::CPUEventType' >>> scons: *** [build/ALPHA_SE/cpu/inorder/resource_pool.do] Error 1 >>> scons: building terminated because of errors. >>> >>> please suggest a way to toggle Werror.... I don-t know from where to >>> toggle it.... I recently downloaded a copy of m5 using mercurial(m5 >>> and not m5-stable). Please advise what to do... >>> >>> -- >>> Sudhanshu([email protected], [email protected] & >>> [email protected]) >>> _______________________________________________ >>> m5-users mailing list >>> [email protected] >>> http://m5sim.org/cgi-bin/mailman/listinfo/m5-users >>> >>> >>> _______________________________________________ >>> m5-users mailing list >>> [email protected] >>> http://m5sim.org/cgi-bin/mailman/listinfo/m5-users >>> >>> >>> -- >>> Sudhanshu([email protected], [email protected] & >>> [email protected]) >>> _______________________________________________ >>> m5-users mailing list >>> [email protected] >>> http://m5sim.org/cgi-bin/mailman/listinfo/m5-users >>> >>> >> >> _______________________________________________ >> m5-users mailing list >> [email protected] >> http://m5sim.org/cgi-bin/mailman/listinfo/m5-users >> >> > > > > _______________________________________________ m5-users mailing list [email protected] http://m5sim.org/cgi-bin/mailman/listinfo/m5-users
