Fixed.
Ali On 11.01.2012 18:12, Andreas Hansson wrote: > Hi guys, > > After applying the changeset that adds support for the partially valid packets, compilation with llvm-gcc fails due to: > > build/ALPHA_SE/mem/packet.cc: In member function 'bool Packet::checkFunctional(Printable*, Addr, int, uint8_t*)': > build/ALPHA_SE/mem/packet.cc:290: warning: comparison is always true due to limited range of data type > scons: *** [build/ALPHA_SE/mem/packet.o] Error 1 > scons: building terminated because of errors. > > The reason is that bytesValidStart is an unsigned and thus always >= 0. I'd suggest to simply remove this part of the check. > > The constructor in packet.hh is also 80+ characters due to the new members (but that is obviously not a big issue). > > Andreas > > -- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. > > _______________________________________________ > 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
