[PATCH] D28952: [analyzer] Add new Z3 constraint manager backend

delcypher added a comment. I'm a little late to the review but hopefully I'll have useful comments. Nice work @ddc > I looked around for a generic smt-lib interface earlier, but couldn't find > much available, and since I wanted floating-point arithmetic support, I ended > up just directly

[PATCH] D28952: [analyzer] Add new Z3 constraint manager backend

delcypher added inline comments. Comment at: CMakeLists.txt:188 +find_package(Z3 4.5) + @ddcc It is of course up to you but I recently [[ added support for using `libz3` directly | added support for using `libz3` directly ]] from CMake. via it's own CMake

[PATCH] D28952: [analyzer] Add new Z3 constraint manager backend

delcypher added a comment. In https://reviews.llvm.org/D28952#655337, @dcoughlin wrote: > In https://reviews.llvm.org/D28952#655278, @ddcc wrote: > > > > - That said, I think there are still significant optimization > > > opportunities. It looks like when performing a query you create a new >

[PATCH] D28952: [analyzer] Add new Z3 constraint manager backend

delcypher added inline comments. Comment at: CMakeLists.txt:188 +find_package(Z3 4.5) + delcypher wrote: > @ddcc It is of course up to you but I recently [[ added support for using > `libz3` directly | added support for using `libz3` directly ]] from CMake.

[PATCH] D28952: [analyzer] Add new Z3 constraint manager backend

delcypher added inline comments. Comment at: lib/StaticAnalyzer/Core/Z3ConstraintManager.cpp:559 + Float.toString(Chars, 0, 0); + AST = Z3_mk_numeral(Z3Context::ZC, Chars.c_str(), Sort); + break; ddcc wrote: > delcypher wrote: > > @ddcc I'm not

[PATCH] D28952: [analyzer] Add new Z3 constraint manager backend

delcypher added inline comments. Comment at: lib/StaticAnalyzer/Core/Z3ConstraintManager.cpp:674 + // are the same size. + Z3_get_numeral_uint64(Z3Context::ZC, AST, +reinterpret_cast<__uint64 *>()); ddcc wrote: > The only

[PATCH] D28952: [analyzer] Add new Z3 constraint manager backend

delcypher added inline comments. Comment at: cmake/modules/FindZ3.cmake:5 +PKG_CHECK_MODULES(PC_Z3 QUIET libz3) +set(Z3_DEFINITIONS ${PC_LIBZ3_CFLAGS_OTHER}) + ddcc wrote: > delcypher wrote: > > @ddcc This CMake variable is set but never used. Also based on the

[PATCH] D28952: [analyzer] Add new Z3 constraint manager backend

delcypher added inline comments. Comment at: cmake/modules/FindZ3.cmake:3 +# in the find_path() and find_library() calls +find_package(PkgConfig QUIET) +PKG_CHECK_MODULES(PC_Z3 QUIET libz3) @ddcc Seeing as you don't want to use the new upstream Z3 CMake package

[PATCH] D28952: [analyzer] Add new Z3 constraint manager backend

delcypher added inline comments. Comment at: CMakeLists.txt:188 +find_package(Z3 4.5) + ddcc wrote: > delcypher wrote: > > delcypher wrote: > > > @ddcc It is of course up to you but I recently [[ added support for using > > > `libz3` directly | added support

[PATCH] D51239: [ubsan] Enable -fsanitize=vptr on Apple devices and simulators

delcypher requested changes to this revision. delcypher added inline comments. This revision now requires changes to proceed. Comment at: clang/lib/Driver/ToolChains/Darwin.cpp:2254 Res |= SanitizerKind::Function; + if (!isTargetMacOS() || !isMacosxVersionLT(10, 9)) +Res

[PATCH] D51239: [ubsan] Enable -fsanitize=vptr on Apple devices and simulators

delcypher accepted this revision. delcypher added a comment. This revision is now accepted and ready to land. LGTM. Comment at: clang/test/Driver/fsanitize.c:426 +// RUN: %clang -target arm-apple-ios4 -fsanitize=vptr %s -### 2>&1 | FileCheck %s

[PATCH] D15225: [Driver] Sanitizer support based on runtime library presence

delcypher added inline comments. Comment at: clang/lib/Driver/ToolChains/Darwin.h:425 + /// \return Relative path to the filename for the library + /// containing the sanitizer {@code SanitizerName}. You might want to update that comment to not mention

[PATCH] D15225: [Driver] Sanitizer support based on runtime library presence

delcypher added a comment. @george.karpenkov Other than the comment that probably needs updating, LGTM. https://reviews.llvm.org/D15225 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D15225: [Driver] Sanitizer support based on runtime library presence

delcypher added inline comments. Comment at: clang/lib/Driver/ToolChains/Darwin.cpp:2298 SanitizerMask Res = ToolChain::getSupportedSanitizers(); - Res |= SanitizerKind::Address; - Res |= SanitizerKind::Leak; - Res |= SanitizerKind::Fuzzer; - Res |=

[PATCH] D15225: [Driver] Sanitizer support based on runtime library presence

delcypher added inline comments. Comment at: clang/lib/Driver/ToolChains/Darwin.cpp:2298 SanitizerMask Res = ToolChain::getSupportedSanitizers(); - Res |= SanitizerKind::Address; - Res |= SanitizerKind::Leak; - Res |= SanitizerKind::Fuzzer; - Res |=

[PATCH] D15225: [Driver] Sanitizer support based on runtime library presence

delcypher requested changes to this revision. delcypher added a comment. This revision now requires changes to proceed. Seems mostly fine apart from some minor nits. If I'm honest I don't see any reason why this should be Darwin specific. Sure the naming convention and location of the runtime

[PATCH] D53239: [python] [tests] Disable python binding tests when building with LLVM_USE_SANITIZER=Address

delcypher accepted this revision. delcypher added a comment. This revision is now accepted and ready to land. LGTM. But then I would say that because I wrote the patch ... Repository: rC Clang https://reviews.llvm.org/D53239 ___ cfe-commits

[PATCH] D54978: Move the SMT API to LLVM

delcypher added a comment. In D54978#1431430 , @mikhail.ramalho wrote: > Hi all, > > Sorry for the massive delay, but I just updated the `FindZ3` script to > retrieve the version from the lib. I changed it to use `try_run` instead of > `try_compile` so

[PATCH] D54978: Move the SMT API to LLVM

delcypher added a comment. In D54978#1431788 , @ddcc wrote: > The only relevant commit that I can find is > https://github.com/Z3Prover/z3/commit/2cb4223979cc94e2ebc4e49a9e83adbdcd2b6979 > , but it first landed in z3 4.6.0. It looks like it's specific

[PATCH] D54978: Move the SMT API to LLVM

delcypher added a comment. In D54978#1432178 , @ddcc wrote: > In D54978#1431935 , @delcypher wrote: > > > Would one of you be able to file a bug against Z3 to fix this? I am no > > longer in a position to

[PATCH] D71491: [ubsan] Check implicit casts in ObjC for-in statements

delcypher added inline comments. Comment at: compiler-rt/lib/ubsan/ubsan_value.cpp:29 +const char *__ubsan::getObjCClassName(ValueHandle Pointer) { +#if defined(__APPLE__) + // We need to query the ObjC runtime for some information, but do not want The

[PATCH] D71491: [ubsan] Check implicit casts in ObjC for-in statements

delcypher added inline comments. Comment at: clang/lib/Driver/ToolChain.cpp:953 + SanitizerKind::ImplicitConversion | SanitizerKind::Nullability | + SanitizerKind::LocalBounds | SanitizerKind::ObjCCast; if (getTriple().getArch() == llvm::Triple::x86 ||

[PATCH] D71491: [ubsan] Check implicit casts in ObjC for-in statements

delcypher added inline comments. Comment at: compiler-rt/lib/ubsan/ubsan_value.cpp:29 +const char *__ubsan::getObjCClassName(ValueHandle Pointer) { +#if defined(__APPLE__) + // We need to query the ObjC runtime for some information, but do not want vsk wrote: >

[PATCH] D71491: [ubsan] Check implicit casts in ObjC for-in statements

delcypher added inline comments. Comment at: compiler-rt/lib/ubsan/ubsan_value.cpp:29 +const char *__ubsan::getObjCClassName(ValueHandle Pointer) { +#if defined(__APPLE__) + // We need to query the ObjC runtime for some information, but do not want vsk wrote: >

[PATCH] D72875: [clang][cmake] Include generated rst files in html built by docs-clang-html target

delcypher accepted this revision. delcypher added a comment. This revision is now accepted and ready to land. LGTM. Thanks for addressing all the issues I raised. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72875/new/

[PATCH] D71491: [ubsan] Check implicit casts in ObjC for-in statements

delcypher added a comment. @vsk The compiler-rt side seems fine to me but I'm not very familiar with the Clang side of things. @arphaman @jfb @rjmccall any thoughts? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71491/new/ https://reviews.llvm.org/D71491

[PATCH] D72875: [clang][cmake] Include generated rst files in html built by docs-clang-html target

delcypher requested changes to this revision. delcypher added inline comments. This revision now requires changes to proceed. Comment at: clang/docs/CMakeLists.txt:93 +function (gen_rst_file output_file td_option source) + get_filename_component(TABLEGEN_INCLUDE_DIR

[PATCH] D72875: [clang][cmake] Include generated rst files in html built by docs-clang-html target

delcypher added inline comments. Comment at: llvm/cmake/modules/AddSphinxTarget.cmake:33 + if (NOT ARG_SOURCE_DIR) +set(ARG_SOURCE_DIR ${CMAKE_CURRENT_SOURCE_DIR}) + endif() @tstellar I'm not 100% sure about this but I think you probably want

[PATCH] D105489: [compiler-rt] [test] Fix asan symbolize tests on py3.10

delcypher accepted this revision. delcypher added a comment. This revision is now accepted and ready to land. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D105489/new/ https://reviews.llvm.org/D105489 ___ cfe-commits mailing list

[PATCH] D101491: [ASan] Rename `-fsanitize-address-destructor-kind=` to drop the `-kind` suffix.

delcypher added a comment. In D101491#2725348 , @jansvoboda11 wrote: > In D101491#2724025 , @delcypher > wrote: > >> @jansvoboda11 Should I use the the alias feature so that the old >>

[PATCH] D101491: [ASan] Rename `-fsanitize-address-destructor-kind=` to drop the `-kind` suffix.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG2d42b2ee7baf: [ASan] Rename `-fsanitize-address-destructor-kind=` to drop the `-kind` suffix. (authored by delcypher). Repository: rG LLVM

[PATCH] D101491: [ASan] Rename `-fsanitize-address-destructor-kind=` to drop the `-kind` suffix.

delcypher updated this revision to Diff 341596. delcypher added a comment. Rename def too. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101491/new/ https://reviews.llvm.org/D101491 Files: clang/docs/ClangCommandLineReference.rst

[PATCH] D101491: [ASan] Rename `-fsanitize-address-destructor-kind=` to drop the `-kind` suffix.

delcypher added a comment. @jansvoboda11 I'm going to land this patch as is (with your nit fixed). If you would like me to add an alias please follow up with me and I can put up a patch to do that. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D101682: [Driver] Fix `ToolChain::getCompilerRTPath()` to return the correct path on Darwin.

delcypher created this revision. delcypher added reviewers: arphaman, dexonsmith, kubamracek, aralisza, yln. delcypher requested review of this revision. Herald added a project: clang. When the Darwin target triple included the OS version number this would cause `ToolChain::getCompilerRTPath()`

[PATCH] D101682: [Driver] Fix `ToolChain::getCompilerRTPath()` to return the correct path on Darwin.

delcypher updated this revision to Diff 342152. delcypher added a comment. - Support other Apple target triples. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101682/new/ https://reviews.llvm.org/D101682 Files: clang/lib/Driver/ToolChain.cpp

[PATCH] D101682: [Driver] Fix `ToolChain::getCompilerRTPath()` to return the correct path on Darwin.

delcypher marked an inline comment as done. delcypher added inline comments. Comment at: clang/lib/Driver/ToolChain.cpp:401 return "aix"; + case llvm::Triple::Darwin: +return "darwin"; arphaman wrote: > Should this also apply to `llvm::Triple::MacOSX`

[PATCH] D101490: [NFC] Rename SanitizeAddressDtorKind codegen opt to not have `Kind` suffix.

delcypher created this revision. delcypher added a reviewer: vitalybuka. Herald added subscribers: dexonsmith, dang. delcypher requested review of this revision. Herald added a project: clang. This is post commit follow up based on discussions in https://reviews.llvm.org/D101122. Repository:

[PATCH] D101122: introduce flag -fsanitize-address-detect-stack-use-after-return-mode. No functional change.

delcypher added inline comments. Comment at: clang/include/clang/Basic/CodeGenOptions.def:225-228 +ENUM_CODEGENOPT(SanitizeAddressDetectStackUseAfterReturnMode, +llvm::AsanDetectStackUseAfterReturnMode, 2, +

[PATCH] D101491: [ASan] Rename `-fsanitize-address-destructor-kind=` to drop the `-kind` suffix.

delcypher created this revision. delcypher added reviewers: vitalybuka, jansvoboda11. Herald added a subscriber: dang. delcypher requested review of this revision. Herald added a project: clang. Renaming the option is based on discussions in https://reviews.llvm.org/D101122. It is normally not a

[PATCH] D101491: [ASan] Rename `-fsanitize-address-destructor-kind=` to drop the `-kind` suffix.

delcypher added a comment. @jansvoboda11 Should I use the the alias feature so that the old `-fsanitize-address-destructor-kind` argument is still parsed? I'm not sure if it's worth it but if doing so is very simple and has a low maintenance burden we should probably do it. Repository: rG

[PATCH] D101491: [ASan] Rename `-fsanitize-address-destructor-kind=` to drop the `-kind` suffix.

delcypher updated this revision to Diff 341384. delcypher added a comment. Remove metavar Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101491/new/ https://reviews.llvm.org/D101491 Files: clang/docs/ClangCommandLineReference.rst

[PATCH] D101490: [NFC] Rename SanitizeAddressDtorKind codegen opt to not have `Kind` suffix.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG1bbbcff99de8: [NFC] Rename SanitizeAddressDtorKind codegen opt to not have `Kind` suffix. (authored by delcypher). Repository: rG LLVM Github

[PATCH] D101682: [Driver] Fix `ToolChain::getCompilerRTPath()` to return the correct path on Apple platforms.

delcypher marked an inline comment as done. delcypher added inline comments. Comment at: clang/lib/Driver/ToolChain.cpp:408 default: return getOS(); } arphaman wrote: > It might be cleaner to do a check for `isOSDarwin` here, as that will help > with

[PATCH] D101682: [Driver] Fix `ToolChain::getCompilerRTPath()` to return the correct path on Apple platforms.

delcypher updated this revision to Diff 342809. delcypher edited the summary of this revision. delcypher added a comment. Use `isOSDarwin()` instead of explicitly checking Darwin OS enum values. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D101682: [Driver] Fix `ToolChain::getCompilerRTPath()` to return the correct path on Apple platforms.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG1971823ecb9e: [Driver] Fix `ToolChain::getCompilerRTPath()` to return the correct path on… (authored by delcypher). Repository: rG LLVM Github

[PATCH] D101122: introduce flag -fsanitize-address-detect-stack-use-after-return-mode. No functional change.

delcypher added inline comments. Comment at: clang/include/clang/Basic/CodeGenOptions.def:225-228 +ENUM_CODEGENOPT(SanitizeAddressDetectStackUseAfterReturnMode, +llvm::AsanDetectStackUseAfterReturnMode, 2, +

[PATCH] D98291: [compiler-rt] Fix stale incremental builds when using `LLVM_BUILD_EXTERNAL_COMPILER_RT=ON`.

delcypher added a comment. In D98291#2616865 , @kubamracek wrote: > Looks good! > > By the way, Apple Clang releases also build compiler-rt this way, so it would > be worth checking with @arphaman that he's okay with the change. @kubamracek @arphaman Ah

[PATCH] D98291: [compiler-rt] Fix stale incremental builds when using `LLVM_BUILD_EXTERNAL_COMPILER_RT=ON`.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGa159f91c8d06: [compiler-rt] Fix stale incremental builds when using… (authored by delcypher). Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D98291: [compiler-rt] Fix stale incremental builds when using `LLVM_BUILD_EXTERNAL_COMPILER_RT=ON`.

delcypher updated this revision to Diff 329469. delcypher added a comment. - update description Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98291/new/ https://reviews.llvm.org/D98291 Files: clang/runtime/CMakeLists.txt Index:

[PATCH] D98291: [compiler-rt] Fix stale incremental builds when using `LLVM_BUILD_EXTERNAL_COMPILER_RT=ON`.

delcypher created this revision. delcypher added reviewers: kubamracek, yln, aralisza, beanz, samsonov. Herald added subscribers: mgorny, dberris. delcypher requested review of this revision. Herald added a project: clang. When building with `LLVM_BUILD_EXTERNAL_COMPILER_RT=ON` (e.g. Swift does

[PATCH] D98291: [compiler-rt] Fix stale incremental builds when using `LLVM_BUILD_EXTERNAL_COMPILER_RT=ON`.

delcypher added a comment. In D98291#2615590 , @kubamracek wrote: > When using Ninja, does this mean running a null build is no longer a null > build, but it at least always invokes this sub-build? Not saying that's bad, > just want to understand the

[PATCH] D98291: [compiler-rt] Fix stale incremental builds when using `LLVM_BUILD_EXTERNAL_COMPILER_RT=ON`.

delcypher added a comment. In D98291#2615593 , @beanz wrote: > So... this patch is fine. (marked as approved). That said, we should really > be working to remove the `LLVM_BUILD_EXTERNAL_COMPILER_RT` option entirely in > favor of just using

[PATCH] D96572: [Clang][ASan] Introduce `-fsanitize-address-destructor-kind=` driver & frontend option.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG5d64dd8e3c22: [Clang][ASan] Introduce `-fsanitize-address-destructor-kind=` driver frontend… (authored by delcypher). Repository: rG LLVM

[PATCH] D97327: [NFC] Switch to auto marshalling infrastructure for `-fsanitize-address-destructor-kind=` flag.

delcypher updated this revision to Diff 326474. delcypher added a comment. - Fix build. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97327/new/ https://reviews.llvm.org/D97327 Files: clang/include/clang/Driver/Options.td

[PATCH] D97327: [NFC] Switch to auto marshalling infrastructure for `-fsanitize-address-destructor-kind=` flag.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG7b1d2a2891d8: [NFC] Switch to auto marshalling infrastructure for `-fsanitize-address… (authored by delcypher). Repository: rG LLVM Github

[PATCH] D96573: [Clang][ASan] Teach Clang to not emit ASan module destructors when compiling with `-mkernel` or `-fapple-kext`.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGfdce098b49cb: [Clang][ASan] Teach Clang to not emit ASan module destructors when compiling… (authored by delcypher). Repository: rG LLVM Github

[PATCH] D97327: [NFC] Switch to auto marshalling infrastructure for `-fsanitize-address-destructor-kind=` flag.

delcypher added a comment. In D97327#2585410 , @jansvoboda11 wrote: > LGTM. > > If you don't get around committing this today, please rebase this on top of > D97375 that I'm going to commit tomorrow. @jansvoboda11

[PATCH] D97327: [NFC] Switch to auto marshalling infrastructure for `-fsanitize-address-destructor-kind=` flag.

delcypher added a comment. In D97327#2582981 , @yln wrote: > LGTM, cleanup only, right? Yes. This is just a small clean up. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97327/new/

[PATCH] D97496: [Clang][ASan] Correct AsanDtorKindToString to return non-void in default case

delcypher added a comment. @cryptoad Thanks. LGTM Originally this code had a `default` case in the switch but clang complained about it being unnecessary because it was exhaustive, so I removed it... It seems this is causing problems with your compiler :( Repository: rG LLVM Github

[PATCH] D96572: [Clang][ASan] Introduce `-fsanitize-address-destructor-kind=` driver & frontend option.

delcypher updated this revision to Diff 325872. delcypher added a comment. Rebase and fix merge conflict caused by f2f59d2a060788f17040ad924ee2d11da0e215c8 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D96572: [Clang][ASan] Introduce `-fsanitize-address-destructor-kind=` driver & frontend option.

delcypher added a comment. In D96572#2563707 , @jansvoboda11 wrote: > In D96572#2561216 , @delcypher wrote: > >> @jansvoboda11 I noticed that the `Options.td` file is making use of some >> fancy mixins that mean

[PATCH] D97327: [NFC] Switch to auto marshalling infrastructure for `-fsanitize-address-destructor-kind=` flag.

delcypher created this revision. delcypher added reviewers: jansvoboda11, arphaman, kubamracek, yln, aralisza. Herald added a subscriber: dang. delcypher requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This change simplifies

[PATCH] D96572: [Clang][ASan] Introduce `-fsanitize-address-destructor-kind=` driver & frontend option.

delcypher added a comment. @arphaman @jansvoboda11 ping. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96572/new/ https://reviews.llvm.org/D96572 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D100087: Include `count` in AppleClang toolchains.

This revision was automatically updated to reflect the committed changes. Closed by commit rGf66e05a720f7: Include `count` in AppleClang toolchains. (authored by delcypher). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100087/new/

[PATCH] D100086: Include `llvm-config` and `not` in AppleClang toolchains.

delcypher created this revision. delcypher added reviewers: JDevlieghere, arphaman. Herald added a subscriber: mgorny. delcypher requested review of this revision. Herald added a project: clang. The motivation here is so that we can configure and run compiler-rt tests from a standalone build

[PATCH] D100086: Include `llvm-config` and `not` in AppleClang toolchains.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG73cbc7f60ed9: Include `llvm-config` and `not` in AppleClang toolchains. (authored by delcypher). Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D100087: Include `count` in AppleClang toolchains.

delcypher created this revision. delcypher added reviewers: arphaman, JDevlieghere. Herald added a subscriber: mgorny. delcypher requested review of this revision. Herald added a project: clang. The motivation here is so we can run the compiler-rt tests from a standalone build against AppleClang.

[PATCH] D100405: Ship `llvm-cxxfilt` in the toolchain.

delcypher created this revision. delcypher added reviewers: steven_wu, arphaman, dexonsmith. Herald added subscribers: yaxunl, mgorny. delcypher requested review of this revision. Herald added a project: clang. Originally done for rdar://problem/57155465. rdar://76602859 Repository: rG LLVM

[PATCH] D100405: Ship `llvm-cxxfilt` in the toolchain.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG4c0bc69490a5: Ship `llvm-cxxfilt` in the toolchain. (authored by delcypher). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D96572: [Clang][ASan] Introduce `-fsanitize-address-destructor-kind=` driver & frontend option.

delcypher added a comment. Note this patch depends on https://reviews.llvm.org/D96571 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96572/new/ https://reviews.llvm.org/D96572 ___ cfe-commits mailing

[PATCH] D96572: [Clang][ASan] Introduce `-fsanitize-address-destructor-kind=` driver & frontend option.

delcypher added inline comments. Comment at: clang/include/clang/Driver/Options.td:1485 +def sanitize_address_destructor_kind_EQ : Joined<["-"], "fsanitize-address-destructor-kind=">, + MetaVarName<"">, + Flags<[CC1Option]>, vitalybuka wrote: > What is the

[PATCH] D96572: [Clang][ASan] Introduce `-fsanitize-address-destructor-kind=` driver & frontend option.

delcypher updated this revision to Diff 323455. delcypher added a comment. Add documentation for `-fsanitize-address-destructor-kind=` option. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96572/new/ https://reviews.llvm.org/D96572 Files:

[PATCH] D96572: [Clang][ASan] Introduce `-fsanitize-address-destructor-kind=` driver & frontend option.

delcypher added inline comments. Comment at: clang/include/clang/Driver/Options.td:1485 +def sanitize_address_destructor_kind_EQ : Joined<["-"], "fsanitize-address-destructor-kind=">, + MetaVarName<"">, + Flags<[CC1Option]>, delcypher wrote: > vitalybuka

[PATCH] D96572: [Clang][ASan] Introduce `-fsanitize-address-destructor-kind=` driver & frontend option.

delcypher added inline comments. Comment at: clang/include/clang/Driver/Options.td:1485 +def sanitize_address_destructor_kind_EQ : Joined<["-"], "fsanitize-address-destructor-kind=">, + MetaVarName<"">, + Flags<[CC1Option]>, vitalybuka wrote: > delcypher

[PATCH] D96572: [Clang][ASan] Introduce `-fsanitize-address-destructor-kind=` driver & frontend option.

delcypher added a comment. @jansvoboda11 I noticed that the `Options.td` file is making use of some fancy mixins that mean the option gets automatically marshalled by the frontend into the codegen options :). I tried this out using the patch to this patch and all my tests seem to pass. Should

[PATCH] D96573: [Clang][ASan] Teach Clang to not emit ASan module destructors when compiling with `-mkernel` or `-fapple-kext`.

delcypher created this revision. delcypher added reviewers: arphaman, kubamracek, yln, aralisza, kcc, vitalybuka. delcypher requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. rdar://71609176 Repository: rG LLVM Github Monorepo

[PATCH] D96572: [Clang][ASan] Introduce `-fsanitize-address-destructor-kind=` driver & frontend option.

delcypher created this revision. delcypher added reviewers: arphaman, kubamracek, yln, aralisza, kcc, vitalybuka. Herald added subscribers: dexonsmith, dang. Herald added a reviewer: jansvoboda11. delcypher requested review of this revision. Herald added a project: clang. Herald added a

[PATCH] D96572: [Clang][ASan] Introduce `-fsanitize-address-destructor-kind=` driver & frontend option.

delcypher updated this revision to Diff 323213. delcypher added a comment. clang-format fix Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96572/new/ https://reviews.llvm.org/D96572 Files: clang/include/clang/Basic/CodeGenOptions.def

[PATCH] D116635: Add warning to detect when calls passing arguments are made to functions without prototypes.

delcypher created this revision. delcypher added reviewers: arphaman, rapidsna, fcloutier, NoQ. delcypher requested review of this revision. Herald added a project: clang. The warning is currently disabled by default but can be enabled with `-Wstrict-calls-without-prototype`. Enabling it by

[PATCH] D116635: Add warning to detect when calls passing arguments are made to functions without prototypes.

delcypher updated this revision to Diff 397431. delcypher added a comment. Add another test case for function definitions without prototypes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116635/new/ https://reviews.llvm.org/D116635 Files:

[PATCH] D116635: Add warning to detect when calls passing arguments are made to functions without prototypes.

delcypher added a comment. Patch https://reviews.llvm.org/D116636 demonstrates the changes needed if the warning was enabled by default: Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116635/new/ https://reviews.llvm.org/D116635

[PATCH] D116635: Add warning to detect when calls passing arguments are made to functions without prototypes.

delcypher added inline comments. Comment at: clang/test/Sema/warn-calls-without-prototype.c:39 + return a + b +c; +} + @NoQ Any ideas about this? It seems kind of weird that when merging `not_a_prototype3` prototype with the K style definition of

[PATCH] D116636: [WIP] Enable `-Wstrict-calls-without-prototype` by default

delcypher added a comment. This change depends on https://reviews.llvm.org/D116635. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116636/new/ https://reviews.llvm.org/D116636 ___ cfe-commits mailing

[PATCH] D116636: [WIP] Enable `-Wstrict-calls-without-prototype` by default

delcypher created this revision. delcypher added reviewers: arphaman, rapidsna, fcloutier, NoQ. delcypher requested review of this revision. Herald added a project: clang. This patch enables the `-Wstrict-calls-without-prototype` warning by default and fixes the existing clang test suite to pass.

[PATCH] D116635: Add warning to detect when calls passing arguments are made to functions without prototypes.

delcypher added inline comments. Comment at: clang/test/Sema/warn-calls-without-prototype.c:39 + return a + b +c; +} + delcypher wrote: > @NoQ Any ideas about this? It seems kind of weird that when merging > `not_a_prototype3` prototype with the K style

[PATCH] D116635: Add warning to detect when calls passing arguments are made to functions without prototypes.

delcypher added inline comments. Comment at: clang/test/Sema/warn-calls-without-prototype.c:39 + return a + b +c; +} + delcypher wrote: > delcypher wrote: > > @NoQ Any ideas about this? It seems kind of weird that when merging > > `not_a_prototype3` prototype

[PATCH] D116635: Add warning to detect when calls passing arguments are made to functions without prototypes.

delcypher added inline comments. Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:5530 + "calling function %0 with arguments when function has no prototype">, InGroup< + DiagGroup<"strict-calls-without-prototype">>, DefaultIgnore; def

[PATCH] D121327: Lower `@llvm.global_dtors` using `__cxa_atexit` on MachO

delcypher added inline comments. Comment at: llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp:1181 +unsigned Priority, const MCSymbol *KeySym) const { + // TODO(yln): Remove -lower-global-dtors-via-cxa-atexit fallback flag + // (LowerGlobalDtorsViaCxaAtExit) and issue a

[PATCH] D116635: Add warning to detect when calls passing arguments are made to functions without prototypes.

delcypher abandoned this revision. delcypher added inline comments. Herald added a project: All. Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:5529 +def warn_call_function_without_prototype : Warning< + "calling function %0 with arguments when function has no

[PATCH] D118855: [modules] Add a flag for TagDecl if it was a definition demoted to a declaration.

delcypher added a comment. Change seems reasonable but I don't have expertise on this code. I've left a few minor nits. Comment at: clang/include/clang/AST/Decl.h:3494 + /// symbol and not interested in the final AST with deduplicated definitions. + bool

[PATCH] D116635: Add warning to detect when calls passing arguments are made to functions without prototypes.

delcypher added inline comments. Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:5529 +def warn_call_function_without_prototype : Warning< + "calling function %0 with arguments when function has no prototype">, InGroup< +

[PATCH] D116635: Add warning to detect when calls passing arguments are made to functions without prototypes.

delcypher added inline comments. Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:5529 +def warn_call_function_without_prototype : Warning< + "calling function %0 with arguments when function has no prototype">, InGroup< +

[PATCH] D116635: Add warning to detect when calls passing arguments are made to functions without prototypes.

delcypher added inline comments. Comment at: clang/test/Sema/warn-calls-without-prototype.c:39 + return a + b +c; +} + aaron.ballman wrote: > delcypher wrote: > > NoQ wrote: > > > delcypher wrote: > > > > delcypher wrote: > > > > > delcypher wrote: > > > > > >

[PATCH] D116635: Add warning to detect when calls passing arguments are made to functions without prototypes.

delcypher added inline comments. Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:5529 +def warn_call_function_without_prototype : Warning< + "calling function %0 with arguments when function has no prototype">, InGroup< +

[PATCH] D116635: Add warning to detect when calls passing arguments are made to functions without prototypes.

delcypher added inline comments. Comment at: clang/test/Sema/warn-calls-without-prototype.c:39 + return a + b +c; +} + NoQ wrote: > delcypher wrote: > > delcypher wrote: > > > delcypher wrote: > > > > @NoQ Any ideas about this? It seems kind of weird that when

[PATCH] D117924: [compiler_rt] Add a seperate runtime for Mac Catalyst

delcypher added a comment. @bc-lee Thanks for the patch. While I get what you're trying to do I have doubts about being able to accept the patch in its current form. Apple's ASan catalyst doesn't work by building a separate dylib, instead it builds the macosx dylib in a different way to make

[PATCH] D124054: [NFC] Avoid unnecessary duplication of code generating diagnostic.

delcypher added a comment. @aaron.ballman Thanks. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124054/new/ https://reviews.llvm.org/D124054 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D124054: [NFC] Avoid unnecessary duplication of code generating diagnostic.

This revision was automatically updated to reflect the committed changes. Closed by commit rG3d612a930dce: [NFC] Avoid unnecessary duplication of code generating diagnostic. (authored by delcypher). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D124054: [NFC] Avoid unnecessary duplication of code generating diagnostic.

delcypher created this revision. delcypher added reviewers: rapidsna, fcloutier, aaron.ballman. Herald added a project: All. delcypher requested review of this revision. Herald added a project: clang. The previous code unneccessarily duplicated the creation of a diagnostic where the only

[PATCH] D124474: Honor COMPILER_RT_INCLUDE_TESTS when using LLVM_BUILD_EXTERNAL_COMPILER_RT=ON

delcypher added a comment. > I'd definitely prefer moving towards LLVM_ENABLE_RUNTIMES. We already require > LLVM_ENABLE_RUNTIMES for libc++, libc++abi and libunwind and I'm going to > propose doing the same for compiler-rt as well. +1 on this. Compiler-RT's CMake code is extremely and

  1   2   >