[PATCH] D56969: GlobalISel: Fix widenScalarToNextPow2 mutation for vectors

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 29 14:30:29 PST 2019


arsenm added a comment.
Herald added a subscriber: Petar.Avramovic.

In D56969#1373222 <https://reviews.llvm.org/D56969#1373222>, @aemerson wrote:

> My preference here would be to have a separate set of mutations for vector elements, @dsanders @aditya_nandakumar what do you think?


What about having 3 versions for each of these? e.g. minScalar, minElement, minScalarOrElement?


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

https://reviews.llvm.org/D56969





More information about the llvm-commits mailing list