kernel: use const reference for SigSet too
authorEddie Hung <eddie@fpgeh.com>
Tue, 17 Mar 2020 17:22:33 +0000 (10:22 -0700)
committerEddie Hung <eddie@fpgeh.com>
Tue, 17 Mar 2020 17:22:33 +0000 (10:22 -0700)
kernel/sigtools.h

index 37674df8165c9fecc036e6ade861539aeccfc7fe..10b39a89e19d3537677daeb73967e97ddf971a5c 100644 (file)
@@ -155,65 +155,65 @@ struct SigSet
 
        void insert(const RTLIL::SigSpec &sig, T data)
        {
-               for (auto &bit : sig)
+               for (const auto &bit : sig)
                        if (bit.wire != NULL)
                                bits[bit].insert(data);
        }
 
-       void insert(RTLIL::SigSpec sig, const std::set<T> &data)
+       void insert(const RTLIL::SigSpec& sig, const std::set<T> &data)
        {
-               for (auto &bit : sig)
+               for (const auto &bit : sig)
                        if (bit.wire != NULL)
                                bits[bit].insert(data.begin(), data.end());
        }
 
-       void erase(RTLIL::SigSpec sig)
+       void erase(const RTLIL::SigSpec& sig)
        {
-               for (auto &bit : sig)
+               for (const auto &bit : sig)
                        if (bit.wire != NULL)
                                bits[bit].clear();
        }
 
-       void erase(RTLIL::SigSpec sig, T data)
+       void erase(const RTLIL::SigSpec &sig, T data)
        {
-               for (auto &bit : sig)
+               for (const auto &bit : sig)
                        if (bit.wire != NULL)
                                bits[bit].erase(data);
        }
 
-       void erase(RTLIL::SigSpec sig, const std::set<T> &data)
+       void erase(const RTLIL::SigSpec &sig, const std::set<T> &data)
        {
-               for (auto &bit : sig)
+               for (const auto &bit : sig)
                        if (bit.wire != NULL)
                                bits[bit].erase(data.begin(), data.end());
        }
 
-       void find(RTLIL::SigSpec sig, std::set<T> &result)
+       void find(const RTLIL::SigSpec &sig, std::set<T> &result)
        {
-               for (auto &bit : sig)
+               for (const auto &bit : sig)
                        if (bit.wire != NULL) {
                                auto &data = bits[bit];
                                result.insert(data.begin(), data.end());
                        }
        }
 
-       void find(RTLIL::SigSpec sig, pool<T> &result)
+       void find(const RTLIL::SigSpec &sig, pool<T> &result)
        {
-               for (auto &bit : sig)
+               for (const auto &bit : sig)
                        if (bit.wire != NULL) {
                                auto &data = bits[bit];
                                result.insert(data.begin(), data.end());
                        }
        }
 
-       std::set<T> find(RTLIL::SigSpec sig)
+       std::set<T> find(const RTLIL::SigSpec &sig)
        {
                std::set<T> result;
                find(sig, result);
                return result;
        }
 
-       bool has(RTLIL::SigSpec sig)
+       bool has(const RTLIL::SigSpec &sig)
        {
                for (auto &bit : sig)
                        if (bit.wire != NULL && bits.count(bit))
@@ -289,14 +289,14 @@ struct SigMap
 
        void add(const RTLIL::SigBit &bit)
        {
-               RTLIL::SigBit b = database.find(bit);
+               const auto &b = database.find(bit);
                if (b.wire != nullptr)
                        database.promote(bit);
        }
 
        void add(const RTLIL::SigSpec &sig)
        {
-               for (auto &bit : sig)
+               for (const auto &bit : sig)
                        add(bit);
        }
 
@@ -335,7 +335,7 @@ struct SigMap
        RTLIL::SigSpec allbits() const
        {
                RTLIL::SigSpec sig;
-               for (auto &bit : database)
+               for (const auto &bit : database)
                        if (bit.wire != nullptr)
                                sig.append(bit);
                return sig;