[PATCH] D59995: [LV] Exclude loop-invariant inputs from scalar cost computation.

Mon Jul 15 01:51:34 PDT 2019

fhahn added a comment.

In D59995#1584967 <https://reviews.llvm.org/D59995#1584967>, @mstorsjo wrote:

> This triggers failed asserts in some cases, reproable with  https://martin.st/temp/loadimage-preproc.cpp.xz, with `clang++ -target i686-w64-mingw32 -c -O3 loadimage-preproc.cpp`. Will post a proper bug report later today, unless someone else beats me to it.

Thanks for reporting the issue, should be fixed in rL366049 <https://reviews.llvm.org/rL366049>.

@Ayal, the problem was that we are not computing the scalarized cost in getVectorIntrinsicCost and TTI::getIntrinsicInstrCost expects the full list of arguments, so we should not filter it. I'll check if there's a reason we do not consider the scalarized cost there.

Repository:
  rL LLVM

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

https://reviews.llvm.org/D59995