Re: [patch] 17/n: trans-mem: compiler trans-mem main engine

2011-11-06 Thread Torvald Riegel
On Fri, 2011-11-04 at 22:48 +0100, Torvald Riegel wrote: > On Thu, 2011-11-03 at 20:38 +, Joseph S. Myers wrote: > > Make sure that you do need each #include present in this and any other new > > file. Since 2008 a lot of includes of tm.h and toplev.h have been removed > > and diagnostic-cor

Re: [patch] 17/n: trans-mem: compiler trans-mem main engine

2011-11-06 Thread Aldy Hernandez
On 11/06/11 07:28, Torvald Riegel wrote: On Fri, 2011-11-04 at 22:48 +0100, Torvald Riegel wrote: On Thu, 2011-11-03 at 20:38 +, Joseph S. Myers wrote: Make sure that you do need each #include present in this and any other new file. Since 2008 a lot of includes of tm.h and toplev.h have be

Re: [patch] 17/n: trans-mem: compiler trans-mem main engine

2011-11-04 Thread Richard Henderson
On 11/04/2011 02:48 PM, Torvald Riegel wrote: > Removed unnecessary includes. > > * trans-mem.c: Removed unnecessary includes. Ok. r~

Re: [patch] 17/n: trans-mem: compiler trans-mem main engine

2011-11-04 Thread Joseph S. Myers
On Fri, 4 Nov 2011, Torvald Riegel wrote: > On Thu, 2011-11-03 at 20:38 +, Joseph S. Myers wrote: > > Make sure that you do need each #include present in this and any other new > > file. Since 2008 a lot of includes of tm.h and toplev.h have been removed > > and diagnostic-core.h introduced

Re: [patch] 17/n: trans-mem: compiler trans-mem main engine

2011-11-04 Thread Torvald Riegel
On Thu, 2011-11-03 at 20:38 +, Joseph S. Myers wrote: > Make sure that you do need each #include present in this and any other new > file. Since 2008 a lot of includes of tm.h and toplev.h have been removed > and diagnostic-core.h introduced as an alternative to diagnostic.h for > many user

Re: [patch] 17/n: trans-mem: compiler trans-mem main engine

2011-11-03 Thread Joseph S. Myers
Make sure that you do need each #include present in this and any other new file. Since 2008 a lot of includes of tm.h and toplev.h have been removed and diagnostic-core.h introduced as an alternative to diagnostic.h for many users. If tm.h is needed, it's good to have a comment on the #includ

[patch] 17/n: trans-mem: compiler trans-mem main engine

2011-11-03 Thread Aldy Hernandez
Here is the main engine (trans-mem.c and trans-mem.h). The file trans-mem.c did not fit within the 100k that the list restricts me to, so I am attaching the patch for both of these files as a bzip2 file. If it is preferable for me to split this one file in two separate posts, I can do so. t