Improved c1visualizer output a bit: Emit a human-readable description for

Improved c1visualizer output a bit: Emit a human-readable description for
changes instead of a bit-pattern. Fixed logic when ranges are emitted. Improved
indentation in hydrogen.cfg (aesthetical change only). Some minor additional
cleanup.

[fschneider, 2011-05-11 08:42:07]

LGTM.

http://codereview.chromium.org/6995024/diff/1/src/hydrogen-instructions.cc
File src/hydrogen-instructions.cc (right):

http://codereview.chromium.org/6995024/diff/1/src/hydrogen-instructions.cc#ne...
src/hydrogen-instructions.cc:423: stream->Add("EVERYTHING");
"*" would be less verbose.

http://codereview.chromium.org/6995024/diff/1/src/hydrogen-instructions.cc#ne...
src/hydrogen-instructions.cc:426: #define DECLARE_DO(type)                      
  \
since it's not a declaration maybe PRINT_DO?

[Sven Panne, 2011-05-11 09:08:25]

http://codereview.chromium.org/6995024/diff/1/src/hydrogen-instructions.cc
File src/hydrogen-instructions.cc (right):

http://codereview.chromium.org/6995024/diff/1/src/hydrogen-instructions.cc#ne...
src/hydrogen-instructions.cc:423: stream->Add("EVERYTHING");
On 2011/05/11 08:42:07, fschneider wrote:
> "*" would be less verbose.

Done.

http://codereview.chromium.org/6995024/diff/1/src/hydrogen-instructions.cc#ne...
src/hydrogen-instructions.cc:426: #define DECLARE_DO(type)                      
  \
On 2011/05/11 08:42:07, fschneider wrote:
> since it's not a declaration maybe PRINT_DO?

Done.

[danno, 2011-05-11 09:10:13]

LGTM

http://codereview.chromium.org/6995024/diff/1/src/hydrogen-instructions.cc
File src/hydrogen-instructions.cc (right):

http://codereview.chromium.org/6995024/diff/1/src/hydrogen-instructions.cc#ne...
src/hydrogen-instructions.cc:63: default:
I think the idea here is to have the compiler complain when something is added
to the enum? With your change, that's no longer the case.