[PATCH] D84324: AMDGPU/GlobalISel: Lower G_FREM

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

I don't think copy the DAG path was necessarily the right choice. The correct thing to do might be to make the DAG path use floor? Is either even correct if this fails conformance?

I don't think this should go in generic code unless we're more sure this is the correct operation


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

https://reviews.llvm.org/D84324