Displaying 2 results from an estimated 2 matches for "r85793".
2009 Nov 10
2
[LLVMdev] Bugfix: SCCP
...ass was failing an assertion that I traced back to
SCCPSolver::visitExtractValueInst. getStructValueState was used on the
aggregate operand even if it had no structure type. I added a check to
use getStructValueState for structs and getValueState for other
operands.
This behavior was introduced in r85793. Please note that I'm not sure
whether my changes correctly address this problem. In case they don't,
I'd be glad to learn why in order to get a better understanding of
llvm. :-)
Please find attached a patch.
Best,
Stephan
-------------- next part --------------
A non-text attachment...
2009 Nov 10
0
[LLVMdev] Bugfix: SCCP
...that I traced back to
> SCCPSolver::visitExtractValueInst. getStructValueState was used on the
> aggregate operand even if it had no structure type. I added a check to
> use getStructValueState for structs and getValueState for other
> operands.
>
> This behavior was introduced in r85793. Please note that I'm not sure
> whether my changes correctly address this problem. In case they don't,
> I'd be glad to learn why in order to get a better understanding of
> llvm. :-)
Thanks, I applied a variant of your patch with a testcase here:
http://lists.cs.uiuc.edu/pip...