This is the mail archive of the java-discuss@sources.redhat.com mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: New C++ ABI: patches.



Oh, actually, I noticed one thing:

+  /* Mangle the class$ field. */
+  for (field = TYPE_FIELDS (type); field; field = TREE_CHAIN (field))
+    if (DECL_NAME (field) == CL_suffix)
+      {
+	name = mangle_decl (field);
+	break;
+      }
+  if (!field)
+    fatal ("Can't find class$");
+

This new code will fire even in the old ABI, and mangle_decl probably
isn't the right thing to use in the old ABI.  Do you want to bracket
this new code with:

  if (!flag_new_abi)

?  You know best -- this code only fires for Java types...

Thanks,

--
Mark Mitchell                   mark@codesourcery.com
CodeSourcery, LLC               http://www.codesourcery.com

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]