[PATCH] D97036: [X86] Fix a codegen crash in getSetCCResultType

Fraser Cormack via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 19 01:26:31 PST 2021


frasercrmck accepted this revision.
frasercrmck added a comment.

In D97036#2574085 <https://reviews.llvm.org/D97036#2574085>, @craig.topper wrote:

> Oh I see. changeVectorElementType assumes that if the type is simple and type to change to is simple it will succeed in MVT. That seems like a poor implementation. Maybe it should check the result and fallback to the extended implementation of it fails on MVT?

Yeah that's.. questionable. I'd +1 that change.

This change, though, LGTM.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97036



More information about the llvm-commits mailing list