[PATCH] D15174: [MSVC] Fix for http://llvm.org/PR25636: indexed accessor property not supported correctly.

[Alexey Bataev via cfe-commits]

Set value is a call to SetX() function, neither the argument of the 
SetX(), nor the result of the GetX(). So, expression a.x=5 always emits 
the following code as a result a.Setx(5). 
MSPropertyRefBuilder::buildSet() does not capture results at all. It 
just ignores

captureSetValueAsResult

value. That's why we have to capture final result after 'set' is built completely.

Best regards,
Alexey Bataev
=============
Software Engineer
Intel Compiler Team

07.12.2015 11:07, John McCall пишет:
> rjmccall added a comment.
>
> I don't understand why that's true.  buildSet is called with captureSetValueAsResult=true, and the set value is definitely capturable, so that value should be set as the result; and you're not overriding it.  Why does the expression end up having type void?
>
>
> http://reviews.llvm.org/D15174
>
>
>