[PATCH] D81297: AMDGPU: Implement computeKnownAlignForTargetInstr

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 17 06:58:42 PDT 2020


arsenm added a comment.

In D81297#2097987 <https://reviews.llvm.org/D81297#2097987>, @foad wrote:

> I'd still be inclined to either follow the precedent of IR, or take the max of the two alignments. But I guess this is OK if you're reluctant to do that.


>From the context of just the intrinsic itself, we don't have the option


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

https://reviews.llvm.org/D81297





More information about the llvm-commits mailing list