[llvm-branch-commits] [llvm] [RISCV] Change RISCVABI::computeTargetABI() to return Expected<ABI> (PR #213410)

[Alexander Richardson via llvm-branch-commits]

================
@@ -29,9 +31,7 @@ RISCVTargetELFStreamer::RISCVTargetELFStreamer(MCStreamer &S,
                                                const MCSubtargetInfo &STI)
     : RISCVTargetStreamer(S), CurrentVendor("riscv") {
   MCAssembler &MCA = getStreamer().getAssembler();
-  auto &MAB = static_cast<RISCVAsmBackend &>(MCA.getBackend());
-  setTargetABI(
-      RISCVABI::computeTargetABI(STI, MAB.getTargetOptions().getABIName()));
+  setTargetABI(static_cast<const RISCVSubtarget &>(STI).getTargetABI());
----------------
arichardson wrote:

Turns out this is not safe since when called from llvm-mc, etc. we don't actually have a RISCVSubtarget.

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