[PATCH] D88806: [SCEV] Model ptrtoint(SCEVUnknown) cast not as unknown, but as zext/trunc/self of SCEVUnknown

Mon Oct 12 10:38:06 PDT 2020

lebedev.ri added a comment.

In D88806#2325340 <https://reviews.llvm.org/D88806#2325340>, @hans wrote:

> Not sure about doom, but it did trigger an assert in Chromium's builds:
>
> llvm/lib/IR/Constants.cpp:1868: static llvm::Constant* llvm::ConstantExpr::getTrunc(llvm::Constant*, llvm::Type*, bool): Assertion `C->getType()->isIntOrIntVectorTy() && "Trunc operand must be integer"' failed.
>
> See https://bugs.chromium.org/p/chromium/issues/detail?id=1137416#c4 for a reproducer.
>
> I've reverted in 17cec6a11a12f815052d56a17ef738cf246a2d9a <https://reviews.llvm.org/rG17cec6a11a12f815052d56a17ef738cf246a2d9a>

Thank you, i can reproduce.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88806/new/

https://reviews.llvm.org/D88806