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

[Dmitri Gribenko]

On Mon, Jan 7, 2013 at 11:38 PM, jahanian <fjahanian at apple.com> wrote:
>
> 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?

Thanks Fariborz!  I read Richard's email incorrectly: I want to remove
it from -ast-print.

My motivation is that I expect -ast-print to print what I wrote in the
source code.

Richard: if this is still useful for you, I will not push this further.

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>*/