[PATCH] D62342: [SelectionDAG] Enhance the simplification of `copyto` from `implicit-def`.
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri May 24 05:38:25 PDT 2019
arsenm added a comment.
In D62342#1515284 <https://reviews.llvm.org/D62342#1515284>, @nhaehnle wrote:
> I don't think this is correct. If an IMPLICIT_DEF has multiple uses, then the correctness of the program may depend on all uses seeing the same value, even if that value is undefined.
I don't think this is a guaranteed property. This is the MIR equivalent of the poison/freeze question, but today implicit_def doesn't give you this
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62342/new/
https://reviews.llvm.org/D62342
More information about the llvm-commits
mailing list