[cfe-commits] [PATCH] Remove 'PrintInstantiation' feature from DeclPrinter

jahanian fjahanian at apple.com
Mon Jan 7 13:38:03 PST 2013


On Jan 7, 2013, at 1:06 PM, Dmitri Gribenko <gribozavr at gmail.com> wrote:

> On Mon, Jan 7, 2013 at 11:05 PM, Richard Trieu <rtrieu at google.com> wrote:
>> I don't understand your reasons for removing this code.  It was specifically
>> added so that template instantiations would be outputted with -ast-dump.
>> Why do you consider printing instantiations misleading?
> 
> It prints something that is not written in the source code.

I think you intend to remove it from -ast-print not -ast-dump. But since it is under an optional print policy
why do we want to remove this capability entirely?

- Fariborz

> 
> Dmitri
> 
> -- 
> main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
> (j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits




More information about the cfe-commits mailing list