[PATCH] D48586: [AMDGPU] Early expansion of 32 bit udiv/urem

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 26 00:14:08 PDT 2018


rampitec added a comment.

In https://reviews.llvm.org/D48586#1143245, @arsenm wrote:

> Probably should add an update_test_checks test for the full expnasions


Probably for scalar cases only. Vectors with scheduler are too unstable. Even scalar will fluctuate with scheduler and RA. Do you think it makes sense to have a separate set of scalar cases?


https://reviews.llvm.org/D48586





More information about the llvm-commits mailing list