commit a338950300adf998b7f1e6993ef403da8b09e1f7
author Cherry Zhang <cherryyz@google.com> Fri Jun 14 16:10:06 2019 -0400
committer Cherry Zhang <cherryyz@google.com> Fri Jun 21 19:55:48 2019 +0000
tree b9174b2544dc7ba4cd2532fca25370fb922d21f1
parent cac8921d9d52458b974149ee0394ff4df601994c

bridge: don't zero temporary variable unnecessarily

If the frontend doesn't ask for zeroing a temporary variable, we
don't need to do that. We need to return a statement, so just
make a dummy one.

Change-Id: Ib5bbb8b6e2e360856400ce6383819e9ca008d709
Reviewed-on: https://go-review.googlesource.com/c/gollvm/+/183265
Reviewed-by: Than McIntosh <thanm@google.com>

bridge/go-llvm.cpp

diff --git a/bridge/go-llvm.cpp b/bridge/go-llvm.cpp
index b779012..bc18b97 100644
--- a/bridge/go-llvm.cpp
+++ b/bridge/go-llvm.cpp
@@ -2299,7 +2299,11 @@
     return tvar;
   }
   tvar->markAsTemporary();
-  Bstatement *is = init_statement(function, tvar, binit);
+  Bstatement *is;
+  if (binit)
+    is = init_statement(function, tvar, binit);
+  else
+    is = statement_list({}); // dummy
   *pstatement = is;
   return tvar;
 }