[llvm] [RISCV] Remove IgnoreUnknown from RISCVISAInfo::parseArchString. (PR #97372)

Pengcheng Wang via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 1 20:16:31 PDT 2024


wangpc-pp wrote:

> Isn't this functionality needed for things like llvm-objdump?..

Some digs into the history:
1.  We added this `IgnoreUnknown` in https://reviews.llvm.org/D139553 to ignore unknown extension when parsing ELF attrs.
2. @asb added `parseNormalizedArchString` in https://reviews.llvm.org/D144353 to do parsing that ignores unknown extensions and removed the only use of `IgnoreUnknown`.

So it seems we don't need `IgnoreUnknown` any more.

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


More information about the llvm-commits mailing list