On Wed, 20 May 2015, Brad Beckmann wrote:
On May 14, 2015, 1:56 p.m., Nilay Vaish wrote:
I think it is not just enabling local variables for action blocks.
My understanding is that this patch allows declaration of local
variables without initialization. My question is would you not
know the initial value or some default value in each case.
Brad Beckmann wrote:
You could make that argument independent of SLICC, yet many programming
languages allow such local variable declarations. This feature simply allows
SLICC to do the same.
On Tues, 5/19 Nilay wrote via email "I agree with you. But I think you
can declare local variables even now, whether initialized or
uninitialized. This change would prevent the warning from getting
printed when the variable is not initialized. So, the patch description
is misleading."
Would it help if we updated the patch description to say:
slicc: remove unitialized local variable warnings in action blocks
This or add a different warning for uninitialized variables.
Either would do for me.
--
Nilay
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev
