[PATCH] D78494: [AMDGPU][CODEGEN] Added 'A' constraint for inline assembler

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 21 08:03:42 PDT 2020


arsenm added a comment.

In D78494#2049119 <https://reviews.llvm.org/D78494#2049119>, @arsenm wrote:

> I guess the fundamental problem for the 16-bit immediate case is you can't know the use instruction's context. You're really accepting something that's not an inline immediate in the use context, but you have no way of knowing that.


Actually, could we emit an error in the MC layer if it turns out to not be used in a 16-bit context? That would make more sense, but we don't have a way of tracking the immediate size


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

https://reviews.llvm.org/D78494





More information about the llvm-commits mailing list