[llvm] RISCVAsmParser: Don't treat operands with relocation specifier as parse-time constants (PR #133377)

[Sam Elliott via llvm-commits]

================
@@ -650,7 +643,7 @@ struct RISCVOperand final : public MCParsedAsmOperand {
     RISCVMCExpr::Specifier VK = RISCVMCExpr::VK_None;
     if (!isImm())
       return false;
-    bool IsConstantImm = evaluateConstantImm(getImm(), Imm, VK);
+    bool IsConstantImm = evaluateConstantImm(getImm(), Imm);
     if (VK == RISCVMCExpr::VK_LO || VK == RISCVMCExpr::VK_PCREL_LO ||
----------------
lenary wrote:

Nit: This `if` is always `false`, and I think `VK` is always none by the time you get to (new) line 654.

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