[PATCH] D153004: [LSR] Consider post-inc form when creating extends/truncates.

Sat Jun 17 09:59:37 PDT 2023

fhahn added a comment.

In D153004#4430394 <https://reviews.llvm.org/D153004#4430394>, @ronlieb wrote:

> seeing this in CI builds of MIOpen.  might be related to this patch. Reproducer might take a few days, so this is mostly FYI
>
> clang++: /jenkins/workspace/compiler-psdb-amd-stg-open_2/repos/external/llvm-project/llvm/lib/Transforms/Scalar/LoopStrengthReduce.cpp:4152: const llvm::SCEV* getAnyExtendConsideringPostIncUses({anonymous}::LSRUse&, const llvm::SCEV*, llvm::Type*, llvm::ScalarEvolution&): Assertion `(!Loops || *Loops == LF.PostIncLoops) && "different post-inc loops used"' failed.4

Thanks for the heads-up. It seems to also trigger during clang bootstrap on some platforms, I'll try to get a reproducer that way. For now I reverted the change

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153004