[PATCH] D73907: [InstCombine] correct operands of shouldChangeType() for casted phi transform

Sat Feb 8 02:18:31 PST 2020

lebedev.ri added subscribers: gchatelet, courbet.
lebedev.ri added a comment.

CC'ing `Align` suspects

In D73907#1865490 <https://reviews.llvm.org/D73907#1865490>, @mstorsjo wrote:

> This causes failed asserts for me. With https://martin.st/temp/substtml-preproc.c, built with `clang -target i686-w64-mingw32 -c -O2 substtml-preproc.c`, I'm getting this:
>
>   clang: ../include/llvm/Support/Alignment.h:384: llvm::Align llvm::operator/(llvm::Align, uint64_t): Assertion `Lhs != 1 && "Can't halve byte alignment"' failed.
>

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73907