This is the mail archive of the systemtap@sourceware.org mailing list for the systemtap project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug translator/10568] translator issue with tapsets


------- Additional Comments From jistone at redhat dot com  2009-08-27 18:39 -------
(In reply to comment #0)
> WARNING: side-effect-free probe 'probe_1809': identifier 'tapset_test' at
> test.stp:1:7
>  source: probe tapset_test.foo { }
>                ^

Using "next" is probably a good filler to make "never" probes be quiet about
being empty.  It might be good to make never probes silently hide though.

As for the rest -- when we resolve a global or function, we add that file to the
session's active files.  The same needs to be done for aliases.  Please see if
the following works for you:

diff --git a/elaborate.cxx b/elaborate.cxx
index bf57f88..17f335d 100644
--- a/elaborate.cxx
+++ b/elaborate.cxx
@@ -540,7 +540,18 @@ alias_expansion_builder
     else
       n->body = new block (alias->body, use->body);
 
+    unsigned old_num_results = finished_results.size();
     derive_probes (sess, n, finished_results, location->optional);
+
+    // Check whether we resolved something. If so, put the
+    // whole library into the queue if not already there.
+    if (finished_results.size() > old_num_results)
+      {
+        stapfile *f = alias->tok->location.file;
+        if (find (sess.files.begin(), sess.files.end(), f)
+            == sess.files.end())
+          sess.files.push_back (f);
+      }
   }
 
   bool checkForRecursiveExpansion (probe *use)


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=10568

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


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