[PATCH] D38546: [ConstantFolding] Avoid assert when folding ptrtoint of vectorized GEP

Bjorn Pettersson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 22 03:58:02 PDT 2017


bjope added a comment.

In https://reviews.llvm.org/D38546#902378, @efriedma wrote:

> Needs a FIXME or something explaining why exactly you're returning early; in theory, we should be able to fold vector ptrtoint operations the same way we fold integer ptrtoint operations, given the right logic.


Ok! I've updated the patch.

> Also, while we're discussing LangRef, would you mind updating it to clarify the rules here?

I've proposed a LangRef update in a separate patch:  https://reviews.llvm.org/D39165


https://reviews.llvm.org/D38546





More information about the llvm-commits mailing list