MaskRay added a comment. In D140690#4018617 <https://reviews.llvm.org/D140690#4018617>, @browneee wrote:
> In D140690#4018043 <https://reviews.llvm.org/D140690#4018043>, @SixWeining > wrote: > >> We notice that DFSan is a work in progress >> <https://clang.llvm.org/docs/DataFlowSanitizer.html#current-status>, >> currently under development for x86_64 Linux. So maybe we can defer the >> change until DFSan development is finished and there is a use case on >> loongarch64? > > This sounds like a reasonable option. > > DFSan development is now fairly mature. We don't support other platforms due > to lack of demand (all the users I'm aware of are x86_64). > > As a side note: I am pleased to see that the patch required to add this > support is as small as I'd expect (even if we add the pieces in D140689 > <https://reviews.llvm.org/D140689> not shown here atm). @browneee Instead of marking tests as `REQUIRES: x86_64-target-arch`, it'd be better to use an opt-out model (`UNSUPPORTED: ...` or `XFAIL: ...`)? The majority of tests should be architecture-agnostic. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140690/new/ https://reviews.llvm.org/D140690 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
