Changeset 2bdf50d

Timestamp:

May 4, 2016, 2:52:07 PM (9 years ago)

Author:

Rob Schluntz <rschlunt@…>

Branches:

ADT, aaron-thesis, arm-eh, ast-experimental, cleanup-dtors, ctor, deferred_resn, demangler, enum, forall-pointer-decay, gc_noraii, jacob/cs343-translation, jenkins-sandbox, master, memory, new-ast, new-ast-unique-expr, new-env, no_list, persistent-indexer, pthread-emulation, qualifiedEnum, resolv-new, with_gc

Children:

99ee64d

Parents:

a1d6d80

Message:

update comment in insertDtors

File:

• src/InitTweak/FixInit.cc (modified) (3 diffs)

src/InitTweak/FixInit.cc

@@ -10 +10 @@
 // Created On       : Wed Jan 13 16:29:30 2016
 // Last Modified By : Rob Schluntz
-// Last Modified On : Mon May 02 14:57:45 2016
+// Last Modified On : Wed May 04 14:51:33 2016
 // Update Count     : 30
 //
@@ -401 +401 @@
         void insertDtors( Iterator begin, Iterator end, OutputIterator out ) {
                 for ( Iterator it = begin ; it != end ; ++it ) {
-                        // remove if instrinsic destructor statement
-                        // xxx - test user manually calling intrinsic functions - what happens?
+                        // remove if instrinsic destructor statement. Note that this is only called
+                        // on lists of implicit dtors, so if the user manually calls an intrinsic
+                        // dtor then the call will still be generated
                         if ( ExprStmt * exprStmt = dynamic_cast< ExprStmt * >( *it ) ) {
                                 ApplicationExpr * appExpr = dynamic_cast< ApplicationExpr * >( exprStmt->get_expr() );
@@ -418 +419 @@
                         } else {
                                 // could also be a compound statement with a loop, in the case of an array
+                                // xxx - write code to remove loop calling intrinsic dtor.
                                 *out++ = (*it)->clone();
                         }