-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 04/05/11 08:51, Nathan Froyd wrote: > On Tue, Apr 05, 2011 at 04:42:27PM +0200, Steven Bosscher wrote: >> On Tue, Apr 5, 2011 at 3:59 PM, Nathan Froyd <froy...@codesourcery.com> >> wrote: >>> v2, now with obstacks! >> >> @findex LOG_LINKS >> @item LOG_LINKS (@var{i}) >> A list (chain of @code{insn_list} expressions) giving information about >> dependencies between instructions within a basic block. Neither a jump >> nor a label may come between the related insns. These are only used by >> the schedulers and by combine. This is a deprecated data structure. >> Def-use and use-def chains are now preferred. > > So, being somewhat RTL-ignorant, is this patch going in the wrong > direction? Should combine be using DF instead of constructing > LOG_LINKS? Ideally, we'd like to get rid of LOG_LINKS in favor of DF; however, I don't think anyone has looked at what would be needed to make that happen for combine.c or at what the memory & compile-time implications might be for such a change.
I still think your patch is a step forward as it should reduce the load on the garbage collector. Jeff -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/ iQEcBAEBAgAGBQJNmz+iAAoJEBRtltQi2kC7NQAH/0GKXSsi3aOoZ/TCYYa2FIpI CfFPLaE9lXfXkFLNIXrVKcWC+NkqbLcFxEQFLusyfQAU4Aqcc0sxFEg69hSCJLPG EWCUBhLqd3YbHpv3pLkykV0nOrd8wBqt24NCbLKaALsNkyvWUGx/hsM3O2lUUbAJ YnUmuyAzx2e5fSQ1WZvfNVb2/GXe7/p0QEDCq/yWf1l/6Pide4EtjWy4NPCbx1C9 AI+P+sqHWwmd6wPzTZTLamOlFCg0QNXwhSJ5eYL0WBkLjuxE9M4EHPiVuyta1Y8c xkCOspGxsq6UdNycM6aGprlHS8uX8O5s4i//lTx/xq5U4n5USKLe3SbLn3Qk9MM= =h8QI -----END PGP SIGNATURE-----