Hi Dave. > Without this patch we generate calls to TM_GETTMCLONE for functions > called indirectly, but we don't actually store the clone mapping in > the clone table because we think the functions are not "needed". > Compiles fine, dies at runtime. See GCC Bugzilla – Bug 53008
Have you taken a look at?: http://gcc.gnu.org/contribute.html In particular, you should include a testcase, and have tested your patch as suggested in the above link. Also, it is customary to include the bugzilla number in the subject as so: Subject: [PR middle-end/53008] trans-mem: make sure clones.... Similarly, the ChangeLog entry should reference "PR middle-end/53008". This makes the final commit to be referenced automagically from the PR itself. All this makes it easier to review your patch. Thanks. Aldy