[PATCH] D78319: [GlobalISel][InlineAsm] Add support for basic input operand constraints

[Konstantin Schwarz via Phabricator via llvm-commits]

kschwarz marked an inline comment as done.
kschwarz added inline comments.


================
Comment at: llvm/lib/CodeGen/GlobalISel/InlineAsmLowering.cpp:574-575
+      LLT Ty = getLLTForType(*Val->getType(), MIRBuilder.getDataLayout());
+      unsigned ExtOpc =
+          Ty.isByteSized() ? TargetOpcode::G_SEXT : TargetOpcode::G_ZEXT;
+      int64_t ExtVal = ExtOpc == TargetOpcode::G_SEXT ? CI->getSExtValue()
----------------
arsenm wrote:
> Changing the extension type based on the byte-sizedness doesn't really make sense to me
I admit I simply followed what is done in the legalizer for extending constants (https://reviews.llvm.org/D70922).

Do you have a suggestion how the condition should look like instead? 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78319