[llvm] r214178 - UseListShuffleVector: Add a copy constructor to appease msc17.

NAKAMURA Takumi geek4civic at gmail.com
Tue Jul 29 15:17:34 PDT 2014


Seems r214224 fixed the issue. Thanks!

2014-07-30 3:37 GMT+09:00 Duncan P. N. Exon Smith <dexonsmith at apple.com>:
>
>> On 2014-Jul-29, at 09:57, David Blaikie <dblaikie at gmail.com> wrote:
>>
>> On Tue, Jul 29, 2014 at 5:20 AM, NAKAMURA Takumi <geek4civic at gmail.com> wrote:
>>> Author: chapuni
>>> Date: Tue Jul 29 07:20:50 2014
>>> New Revision: 214178
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=214178&view=rev
>>> Log:
>>> UseListShuffleVector: Add a copy constructor to appease msc17.
>>
>
> Thanks for fixing this for me!
>
>> Could you mention/describe where copy operations were required? It
>> might be worth fixing that (& if it just needs to /see/ a declaration
>> but never call it, it could be declared-but-not-defined)
>
> I'm interested too.  I'm likely to leave it out again if I don't understand
> why I needed it here.  (I didn't seem to get a buildbot email.)



More information about the llvm-commits mailing list