[llvm-commits] [dragonegg] r138216 - /dragonegg/trunk/src/Constants.cpp

[Duncan Sands]

Author: baldrick
Date: Sun Aug 21 06:41:21 2011
New Revision: 138216

URL: http://llvm.org/viewvc/llvm-project?rev=138216&view=rev
Log:
Add a comment on constant merging.

Modified:
    dragonegg/trunk/src/Constants.cpp

Modified: dragonegg/trunk/src/Constants.cpp
URL: http://llvm.org/viewvc/llvm-project/dragonegg/trunk/src/Constants.cpp?rev=138216&r1=138215&r2=138216&view=diff
==============================================================================
--- dragonegg/trunk/src/Constants.cpp (original)
+++ dragonegg/trunk/src/Constants.cpp Sun Aug 21 06:41:21 2011
@@ -1405,6 +1405,9 @@
   align = CONSTANT_ALIGNMENT(exp, align);
 #endif
   Slot->setAlignment(align);
+  // Allow identical constants to be merged if the user allowed it.
+  // FIXME: maybe this flag should be set unconditionally, and instead the
+  // ConstantMerge pass should be disabled if flag_merge_constants is zero.
   Slot->setUnnamedAddr(flag_merge_constants);
 
   return Slot;