[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