cppcheck

reactorlabs · Apr 23, 2021 · 2b52f8f · 2b52f8f
1 parent 5c170a5
commit 2b52f8f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/rir/src/compiler/gnur2pir/gnur2pir.cpp b/rir/src/compiler/gnur2pir/gnur2pir.cpp
@@ -426,7 +426,7 @@ static void findMerges(const RBCCode& bytecode, CompilerInfo& info) {
 
 struct BCCompiler {
     CompilerInfo& cmp;
-    BCCompiler(CompilerInfo& cmp) : cmp(cmp) {}
+    explicit BCCompiler(CompilerInfo& cmp) : cmp(cmp) {}
 
     void push(Value* v) { cmp.stack.push(v); }
 
@@ -447,29 +447,29 @@ struct BCCompiler {
     Value* env() { return cmp.insert.env; }
 
     template <RBC::Id BC>
-    void compile(RBC);
+    void compile(const RBC&);
 };
 
 // Start instructions translation
 
 template <>
-void BCCompiler::compile<RBC::GETVAR_OP>(RBC bc) {
+void BCCompiler::compile<RBC::GETVAR_OP>(const RBC& bc) {
     auto v = insert(new LdVar(cnst(bc.imm(0)), env()));
     insertPush(new Force(v, env(), Tombstone::framestate()));
 }
 
 template <>
-void BCCompiler::compile<RBC::RETURN_OP>(RBC bc) {
+void BCCompiler::compile<RBC::RETURN_OP>(const RBC& bc) {
     insert(new Return(pop()));
 }
 
 template <>
-void BCCompiler::compile<RBC::LDCONST_OP>(RBC bc) {
+void BCCompiler::compile<RBC::LDCONST_OP>(const RBC& bc) {
     push(insert(new LdConst(cnst(bc.imm(0)))));
 }
 
 template <>
-void BCCompiler::compile<RBC::ADD_OP>(RBC bc) {
+void BCCompiler::compile<RBC::ADD_OP>(const RBC& bc) {
     auto a = pop();
     auto b = pop();
     insertPush(new Add(a, b, env(), bc.imm(0)));

diff --git a/rir/src/compiler/gnur2pir/gnur2pir.h b/rir/src/compiler/gnur2pir/gnur2pir.h
@@ -14,7 +14,7 @@ class Gnur2Pir {
     Module& m;
 
   public:
-    Gnur2Pir(Module& m) : m(m){};
+    explicit Gnur2Pir(Module& m) : m(m){};
     pir::ClosureVersion* compile(SEXP src, const std::string& name);
 };