[llvm] [TextAPI] Add option to filter out unsupported/unknown/invalid targets (PR #195161)

[Zixu Wang via llvm-commits]

================
@@ -313,6 +314,12 @@ template <> struct MappingTraits<UndefinedSection> {
 template <> struct MappingTraits<SymbolSection> {
   static void mapping(IO &IO, SymbolSection &Section) {
     IO.mapRequired("targets", Section.Targets);
+    // With SkipUnknownTriples, ScalarTraits of Target accepts unknown
+    // arch/platform scalars without erroring, leaving invalid Targets in the
+    // vector. Drop them so downstream code only sees valid Targets.
----------------
zixu-w wrote:

That's fair. It would be nice if the YAML facility could offer a cleaner way, but I can see that's a much bigger change.

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