probinson added a comment.
The changes in this patch assume that there aren't any possible suffixes after
the `-zos` part of the triple (no version numbers, like you might find with
darwin or macos, and nothing like `-elf` or `-eabi` like some targets have).
If there are suffixes, I'll happily revise to put `{{.*}}` after everything.
The one test I could not verify is llvm/test/Support/encoding.ll, because I
don't have the utilities that it needs. But `UNSUPPORTED: !` was the
workaround for not having triples allowed in `REQUIRES:` so I think it's the
right change.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139444/new/
https://reviews.llvm.org/D139444
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
