[PATCH] D14122: [SimplifyCFG] Trim duplicate basic blocks in switch cases

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 28 17:17:46 PDT 2015


sanjoy added inline comments.

================
Comment at: include/llvm/IR/BasicBlock.h:318
@@ -317,1 +317,3 @@
 
+  /// Checks if the bodies of two basic blocks are identical.
+  bool isIdenticalTo(const BasicBlock &Other) const;
----------------
dylanmckay wrote:
> The `AUTOBRIEF` Doxygen option was enabled a few months ago - this is no longer necessary and presents visual noise.
> 
> I will however add it if you think consistency is worth it.
> this is no longer necessary and presents visual noise.

SGTM (i.e. it is fine to not add `\brief`).


http://reviews.llvm.org/D14122





More information about the llvm-commits mailing list