[PATCH] D59023: [RISCV] Support -target-abi at the MC layer and for codegen

Ana Pazos via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 8 10:00:42 PST 2019


apazos added inline comments.


================
Comment at: lib/Target/RISCV/RISCVSubtarget.cpp:42
+  if (Expected<RISCVABI::ABI> ABIOrError =
+          RISCVABI::computeTargetABI(TT, getFeatureBits(), ABIName))
+    TargetABI = ABIOrError.get();
----------------
Now it always returns a valid ABI, right? Do we need this check?


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

https://reviews.llvm.org/D59023





More information about the llvm-commits mailing list