[PATCH] [core] add support to resolve section groups.

Shankar Kalpathi Easwaran shankarke at gmail.com
Tue Mar 25 15:35:01 PDT 2014



================
Comment at: lib/Core/Resolver.cpp:219
@@ +218,3 @@
+    // for the first time
+    if (isFirstTime) {
+      for (const Reference *r : atom) {
----------------
Rui Ueyama wrote:
> Is this the correct semantics? If it's not a COMDAT group, we have to report an error for duplicate section group, no?
No. The typeGroupComdat atom has the list of all group child members. 


http://llvm-reviews.chandlerc.com/D3182



More information about the llvm-commits mailing list