[clang] [Clang][Sanitizers] Enable NSAN on X86_64 only (PR #95885)

2024-06-18 Thread Alexander Shaposhnikov via cfe-commits
https://github.com/alexander-shaposhnikov closed https://github.com/llvm/llvm-project/pull/95885 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang][Sanitizers] Enable NSAN on X86_64 only (PR #95885)

2024-06-18 Thread Alexander Shaposhnikov via cfe-commits
alexander-shaposhnikov wrote: yeah, but I kind of wanted to be defensive - people will try the tool and I wanted to avoid any potential confusions / overpromises. https://github.com/llvm/llvm-project/pull/95885 ___ cfe-commits mailing list cfe-commit

[clang] [Clang][Sanitizers] Enable NSAN on X86_64 only (PR #95885)

2024-06-18 Thread Fangrui Song via cfe-commits
https://github.com/MaskRay approved this pull request. https://github.com/llvm/llvm-project/pull/95885 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang][Sanitizers] Enable NSAN on X86_64 only (PR #95885)

2024-06-18 Thread Fangrui Song via cfe-commits
MaskRay wrote: > ~90% of the functionality would work on aarch64, but there are a few > x86_64-specific quirks (__float128, x86-64-specific assumptions about long > double in the helper routines) that need to be refactored (beyond my current > bandwidth). I am happy to investigate them. I gue

[clang] [Clang][Sanitizers] Enable NSAN on X86_64 only (PR #95885)

2024-06-18 Thread Alexander Shaposhnikov via cfe-commits
alexander-shaposhnikov wrote: ~90% of the functionality would work on aarch64, but there are a few x86_64-specific quirks (__float128, x86-64-specific assumptions about long double in the helper routines) that need to be refactored (beyond my current bandwidth). https://github.com/llvm/llvm-p

[clang] [Clang][Sanitizers] Enable NSAN on X86_64 only (PR #95885)

2024-06-18 Thread Fangrui Song via cfe-commits
MaskRay wrote: If the code is generic, it might already work on aarch64... It seems sufficient to avoid compiler-rt runtime testing. https://github.com/llvm/llvm-project/pull/95885 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.

[clang] [Clang][Sanitizers] Enable NSAN on X86_64 only (PR #95885)

2024-06-18 Thread Vitaly Buka via cfe-commits
https://github.com/vitalybuka approved this pull request. https://github.com/llvm/llvm-project/pull/95885 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang][Sanitizers] Enable NSAN on X86_64 only (PR #95885)

2024-06-17 Thread Alexander Shaposhnikov via cfe-commits
https://github.com/alexander-shaposhnikov edited https://github.com/llvm/llvm-project/pull/95885 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang][Sanitizers] Enable NSAN on X86_64 only (PR #95885)

2024-06-17 Thread Alexander Shaposhnikov via cfe-commits
https://github.com/alexander-shaposhnikov edited https://github.com/llvm/llvm-project/pull/95885 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang][Sanitizers] Enable NSAN on X86_64 only (PR #95885)

2024-06-17 Thread Alexander Shaposhnikov via cfe-commits
https://github.com/alexander-shaposhnikov updated https://github.com/llvm/llvm-project/pull/95885 >From 7c286a320256732eca6a9183a1ba21ae60b99746 Mon Sep 17 00:00:00 2001 From: Alexander Shaposhnikov Date: Tue, 18 Jun 2024 06:15:12 + Subject: [PATCH] [Clang][Sanitizers] Enable NSAN on X86_64

[clang] [Clang][Sanitizers] Enable NSAN on X86_64 only (PR #95885)

2024-06-17 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-driver @llvm/pr-subscribers-clang Author: Alexander Shaposhnikov (alexander-shaposhnikov) Changes This is a follow-up to https://github.com/llvm/llvm-project/pull/93783. The current set of patches covers only x86_64 (in particular, on compiler-rt's

[clang] [Clang][Sanitizers] Enable NSAN on X86_64 only (PR #95885)

2024-06-17 Thread Alexander Shaposhnikov via cfe-commits
https://github.com/alexander-shaposhnikov created https://github.com/llvm/llvm-project/pull/95885 This is a follow-up to https://github.com/llvm/llvm-project/pull/93783. The current set of patches covers only x86_64 (in particular, on compiler-rt's side), therefore we should now enable this fla