Re: [PATCH] D11360: Proposed patch to prevent the creation of empty (forwarding) blocks resulting from nested ifs.

[Mehdi AMINI via cfe-commits]

mehdi_amini added inline comments.

================
Comment at: lib/CodeGen/CGStmt.cpp:614
@@ -610,1 +613,3 @@
+      SimplifyForwardingBlocks(CurBlock); 
+  }
 
----------------
Document

================
Comment at: lib/CodeGen/CGStmt.cpp:633
@@ -626,1 +632,3 @@
+      if (CurBlock)
+        SimplifyForwardingBlocks(CurBlock); 
     }
----------------
Document

================
Comment at: test/CodeGen/forwarding-blocks-if.c:17
@@ +16,3 @@
+    return 0;
+}
+
----------------
Any reason to not stick with LLVM coding convention here?


http://reviews.llvm.org/D11360



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits