i965: Mark cfg dumping functions const.
authorKenneth Graunke <kenneth@whitecape.org>
Thu, 4 Sep 2014 06:29:04 +0000 (23:29 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Fri, 5 Sep 2014 16:52:38 +0000 (09:52 -0700)
The dump() methods don't alter the CFG or basic blocks, so we should
mark them as const.  This lets you call them even if you have a const
cfg_t - which is the case in certain portions of the code (such as live
interval handling).

Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Matt Turner <mattst88@gmail.com>
src/mesa/drivers/dri/i965/brw_cfg.cpp
src/mesa/drivers/dri/i965/brw_cfg.h

index 8714b682be0b681c12814f1c9d25a822597a27f7..e67b13a86ef9240f58b9a35aeb749dc2cdd2bf99 100644 (file)
@@ -145,7 +145,7 @@ bblock_t::combine_with(bblock_t *that)
 }
 
 void
-bblock_t::dump(backend_visitor *v)
+bblock_t::dump(backend_visitor *v) const
 {
    int ip = this->start_ip;
    for (backend_instruction *inst = (backend_instruction *)this->start;
@@ -420,7 +420,7 @@ cfg_t::make_block_array()
 }
 
 void
-cfg_t::dump(backend_visitor *v)
+cfg_t::dump(backend_visitor *v) const
 {
    foreach_block (block, this) {
       fprintf(stderr, "START B%d", block->num);
index ca6a2ac2d66e576c4372afd893fa91a6717ecac3..7985a9795aca3ae33a44e0f462105e7d14d05306 100644 (file)
@@ -62,7 +62,7 @@ struct bblock_t {
    bool is_successor_of(const bblock_t *block) const;
    bool can_combine_with(const bblock_t *that) const;
    void combine_with(bblock_t *that);
-   void dump(backend_visitor *v);
+   void dump(backend_visitor *v) const;
 #endif
 
    struct exec_node link;
@@ -100,7 +100,7 @@ struct cfg_t {
    void set_next_block(bblock_t **cur, bblock_t *block, int ip);
    void make_block_array();
 
-   void dump(backend_visitor *v);
+   void dump(backend_visitor *v) const;
 #endif
    void *mem_ctx;