[llvm] r186035 - Add a comment.

Adrian Prantl aprantl at apple.com
Thu Jul 11 14:19:47 PDT 2013


On Jul 10, 2013, at 2:55 PM, David Blaikie <dblaikie at gmail.com> wrote:

> On Wed, Jul 10, 2013 at 2:47 PM, Adrian Prantl <aprantl at apple.com> wrote:
>> 
>> On Jul 10, 2013, at 2:14 PM, David Blaikie <dblaikie at gmail.com> wrote:
>> 
>>> On Wed, Jul 10, 2013 at 2:08 PM, Adrian Prantl <aprantl at apple.com> wrote:
>>>> Author: adrian
>>>> Date: Wed Jul 10 16:08:02 2013
>>>> New Revision: 186035
>>>> 
>>>> URL: http://llvm.org/viewvc/llvm-project?rev=186035&view=rev
>>>> Log:
>>>> Add a comment.
>>> 
>>> Is this the minimal source required to reproduce the issue? Is it
>>> really dependent on templates?
>> 
>> It’s a backend testcase, so obviously the answer is no,
> 
> What I mean is: could we still test this bug with an example that
> doesn't have templates in it. (ie: is the source code minimized)
> 
>> but I’m not convinced that manually instantiating templates in a comment in a test case is the best use of anyone’s time :-)
> 
> Not sure I follow, but maybe: I do think it's a valuable use of
> someone's time to provide simple test cases. The way I'd prefer to see
> debug info test cases simplified is by simplifying the input source,
> not hand-simplifying the IR. This will keep them small, but
> maintainable.

Done in r186119. Did you get a chance to look at my other mail from yesterday?

thanks,
adrian



More information about the llvm-commits mailing list