[PATCH] D91314: [RISCV] ELF attribute for Zfh extension.

[Luís Marques via Phabricator via llvm-commits]

luismarques added inline comments.


================
Comment at: llvm/lib/Object/ELFObjectFile.cpp:335
+      case 'z':
+        if (Arch.consume_front("zfh")) {
+          Features.AddFeature("experimental-zfh");
----------------
Do we need to actually consume the prefix, or would using just `startswith` be fine?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91314