[llvm] [AMDGPU] Add lit() asm operand modifier for SP3 compatibility. (PR #68839)

Stanislav Mekhanoshin via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 12 00:06:02 PDT 2023


================
@@ -3005,7 +3006,7 @@ bool
 AMDGPUAsmParser::isNamedOperandModifier(const AsmToken &Token, const AsmToken &NextToken) const {
   if (Token.is(AsmToken::Identifier) && NextToken.is(AsmToken::LParen)) {
     const auto &str = Token.getString();
-    return str == "abs" || str == "neg" || str == "sext";
----------------
rampitec wrote:

Yeah, we do. Yet another compatibility thing. There is a special handling of the neg(abs()), the only useful combination. If we ever want to handle lit(abs()) it would need a similar special case handling.

https://github.com/llvm/llvm-project/pull/68839


More information about the llvm-commits mailing list