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

Reply via email to