[PATCH] D60793: [Evaluator] Walk initial elements when handling load through bitcast
Robert Lougher via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 18 05:26:38 PDT 2019
rob.lougher added a comment.
In D60793#1471294 <https://reviews.llvm.org/D60793#1471294>, @evgeny777 wrote:
> > what's wrong with keeping it like it is?
>
> Well, if it were possible to avoid the same computation for load I'd like to avoid it. However your patch seems to also handle cases when store is using GEP and load is using bitcast:
>
> So let's stick with current approach. I'd probably include the example above as a test case.
Yes. I will add it as an extra test (after Easter, as I am now on vacation).
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60793/new/
https://reviews.llvm.org/D60793
More information about the llvm-commits
mailing list