On Mon, 1 Sep 2014, Andreas Sandberg wrote:
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/2372/#review5321
-----------------------------------------------------------
I looked into this tool a couple of months ago. At the time, I
encountered some issues relating to indentation (IIRC, the way we indent
case labels by 2 spaces wasn't supported). My conclusion at the time was
that it wasn't possible to configure clang-format to adhere to the gem5
style. Has this changed? If not, I'd suggest that we don't commit this
for now and wait while clang-format matures.
I am not in a hurry to commit the patch. But I don't think waiting for
clang-format to change is the right thing either.
--
Nilay
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev
