[PATCH] D13169: [ELF2] - Implemented --allow-multiple-definition flag

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 28 10:50:40 PDT 2015


ruiu accepted this revision.
ruiu added a comment.
This revision is now accepted and ready to land.

LGTM


================
Comment at: ELF/SymbolTable.cpp:155
@@ +154,3 @@
+                     OldFile->getName() + " and " + NewFile->getName())
+                        .str();
+  if (Config->AllowMultipleDefinition)
----------------
This indentation looked weird at first but that is indeed clang-format would format. So this is what we want.


http://reviews.llvm.org/D13169





More information about the llvm-commits mailing list