[Lldb-commits] [lldb] [lldb] Add SubtargetFeatures to ArchSpec (PR #173046)

[via lldb-commits]

daniilavdeev wrote:

> Arguably, both of these are wrong.

Yes, that is true. However, it seems to me that the extension compatibility issues should be handled at the MCDisassembler level, rather than in LLDB, so for now I believe it would be preferable to be consistent with llvm-objdump output.

> I think your proposed approach (prioritise one of the conflicting extensions, also emit a warning) is acceptable at the moment. Later we can choose which of the three more accurate approaches we prefer (or how to combine them for most accuracy).

Thanks, I will move forward with this approach then.

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