[clang] [clang][driver] add pointer qualifier to auto type (PR #91874)
VoltrexKeyva wrote: Pinging again after another week of no reviews. https://github.com/llvm/llvm-project/pull/91874 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[clang] [clang][driver] add pointer qualifier to auto type (PR #91874)
VoltrexKeyva wrote: Pinging are there haven't been any reviews in a week. https://github.com/llvm/llvm-project/pull/91874 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[clang] [clang][driver] add pointer qualifier to auto type (PR #91874)
https://github.com/VoltrexKeyva created https://github.com/llvm/llvm-project/pull/91874 Fix an LLVM coding standards inconformity by adding pointer qualifiers to `auto`-typed variables that deduce to pointer types. See what the coding standards state [here](https://llvm.org/docs/CodingStandards.html#beware-unnecessary-copies-with-auto). >From 8a09433a80243fb6dd7eb36623bb3df93e862264 Mon Sep 17 00:00:00 2001 From: Mohammed Keyvanzadeh Date: Sun, 12 May 2024 03:13:35 +0330 Subject: [PATCH] [clang][driver] add pointer qualifier to auto type Fix an LLVM coding standards inconformity by adding pointer qualifiers to `auto`-typed variables that deduce to pointer types. --- clang/lib/Driver/Action.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/clang/lib/Driver/Action.cpp b/clang/lib/Driver/Action.cpp index 849bf6035ebd2..e642cde9dca37 100644 --- a/clang/lib/Driver/Action.cpp +++ b/clang/lib/Driver/Action.cpp @@ -246,8 +246,8 @@ void OffloadAction::doOnHostDependence(const OffloadActionWorkTy ) const { void OffloadAction::doOnEachDeviceDependence( const OffloadActionWorkTy ) const { - auto I = getInputs().begin(); - auto E = getInputs().end(); + auto *I = getInputs().begin(); + auto *E = getInputs().end(); if (I == E) return; @@ -261,7 +261,7 @@ void OffloadAction::doOnEachDeviceDependence( if (HostTC) ++I; - auto TI = DevToolChains.begin(); + auto *TI = DevToolChains.begin(); for (; I != E; ++I, ++TI) Work(*I, *TI, (*I)->getOffloadingArch()); } ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[clang] 467bb47 - [clang][cmake] Remove extra brace
Author: Mohammed Keyvanzadeh Date: 2022-09-10T05:00:23+04:30 New Revision: 467bb47c841897450990f3ab82e79fec5f2d1f60 URL: https://github.com/llvm/llvm-project/commit/467bb47c841897450990f3ab82e79fec5f2d1f60 DIFF: https://github.com/llvm/llvm-project/commit/467bb47c841897450990f3ab82e79fec5f2d1f60.diff LOG: [clang][cmake] Remove extra brace Remove the extra trailing brace from the local variable accessor. Differential Revision: https://reviews.llvm.org/D133613 Added: Modified: clang/cmake/modules/AddClang.cmake Removed: diff --git a/clang/cmake/modules/AddClang.cmake b/clang/cmake/modules/AddClang.cmake index 21ac332e4f5fc..d952ff3165587 100644 --- a/clang/cmake/modules/AddClang.cmake +++ b/clang/cmake/modules/AddClang.cmake @@ -70,7 +70,7 @@ macro(add_clang_library name) ${CLANG_SOURCE_DIR}/include/clang/${lib_path}/*.td ) source_group("TableGen descriptions" FILES ${tds}) - set_source_files_properties(${tds}} PROPERTIES HEADER_FILE_ONLY ON) + set_source_files_properties(${tds} PROPERTIES HEADER_FILE_ONLY ON) if(headers OR tds) set(srcs ${headers} ${tds}) ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits