This is an archive of the discontinued LLVM Phabricator instance.

NewGVN: Make sure we don't incorrectly use PredicateInfo when doing PHI of ops
ClosedPublic

Authored by dberlin on Aug 25 2017, 8:22 PM.

Details

Summary

When we backtranslate expressions, we can't use the predicateinfo, since we are evaluating them in a different context.

Diff Detail

Repository
rL LLVM

Event Timeline

dberlin created this revision.Aug 25 2017, 8:22 PM

Add testcase

This revision was automatically updated to reflect the committed changes.
davide added inline comments.Sep 1 2017, 12:24 PM
llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp
1788

Oh, this was where I saw that code.