[clang] [Clang][ARM] support arm target attribute, and warning for bad typo (PR #74812)

2023-12-16 Thread via cfe-commits
https://github.com/hstk30-hw updated https://github.com/llvm/llvm-project/pull/74812 >From dc88d97d00f389b46f51c19a22c17397e1e89b7f Mon Sep 17 00:00:00 2001 From: hstk30-hw Date: Fri, 8 Dec 2023 14:29:33 +0800 Subject: [PATCH] feat: support arm target attribute, and warning for bad typo ---

[clang] [Clang][ARM] support arm target attribute, and warning for bad typo (PR #74812)

2023-12-14 Thread David Spickett via cfe-commits
DavidSpickett wrote: I was mistaken about the `target_link_libraries`, that's what `clangBasic` links to not `clangAST`. It's possible that `clangBasic` now needs to depend on `clangAST`, assuming cmake and the linker are ok with that. https://github.com/llvm/llvm-project/pull/74812

[clang] [Clang][ARM] support arm target attribute, and warning for bad typo (PR #74812)

2023-12-14 Thread David Spickett via cfe-commits
DavidSpickett wrote: Also it seems that `clangAST` links to `clangBasic` but it doesn't `DEPENDS` on them? Not sure what that is trying to achieve. https://github.com/llvm/llvm-project/pull/74812 ___ cfe-commits mailing list

[clang] [Clang][ARM] support arm target attribute, and warning for bad typo (PR #74812)

2023-12-14 Thread David Spickett via cfe-commits
DavidSpickett wrote: Not so sure about that: ``` LINK: command "C:\BuildTools\VC\Tools\MSVC\14.29.30133\bin\Hostx64\x64\link.exe /nologo tools\clang\tools\clang-format\CMakeFiles\clang-format.dir\ClangFormat.cpp.obj

[clang] [Clang][ARM] support arm target attribute, and warning for bad typo (PR #74812)

2023-12-14 Thread via cfe-commits
hstk30-hw wrote: CI fail in `ninja check-clang` build `bin/clang-format.exe` in windows, it has nothing about my code. @gkistanova https://github.com/llvm/llvm-project/pull/74812 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] [Clang][ARM] support arm target attribute, and warning for bad typo (PR #74812)

2023-12-14 Thread via cfe-commits
https://github.com/hstk30-hw updated https://github.com/llvm/llvm-project/pull/74812 >From f9c6d46e73b612c261db5fdfebf49bb28003cf0d Mon Sep 17 00:00:00 2001 From: hstk30-hw Date: Fri, 8 Dec 2023 14:29:33 +0800 Subject: [PATCH] feat: support arm target attribute, and warning for bad typo ---

[clang] [Clang][ARM] support arm target attribute, and warning for bad typo (PR #74812)

2023-12-13 Thread via cfe-commits
https://github.com/hstk30-hw updated https://github.com/llvm/llvm-project/pull/74812 >From b6595d617c6ac0f3e402aeb782eeeb78f90c5fb1 Mon Sep 17 00:00:00 2001 From: hstk30-hw Date: Fri, 8 Dec 2023 14:29:33 +0800 Subject: [PATCH] feat: support arm target attribute, and warning for bad typo ---

[clang] [Clang][ARM] support arm target attribute, and warning for bad typo (PR #74812)

2023-12-07 Thread via cfe-commits
https://github.com/hstk30-hw updated https://github.com/llvm/llvm-project/pull/74812 >From 3e4b7b9da50d742b7bfafa04cdf0c3a64d0d1b9e Mon Sep 17 00:00:00 2001 From: hstk30-hw Date: Fri, 8 Dec 2023 14:29:33 +0800 Subject: [PATCH] feat: support arm target attribute, and warning for bad typo ---

[clang] [Clang][ARM] support arm target attribute, and warning for bad typo (PR #74812)

2023-12-07 Thread via cfe-commits
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff 28a78e2a4a2c358900aaac1a1eb9efce17a7f5a5 8a84eaf11bb76aba7db5243390f246d40b3f9630 --

[clang] [Clang][ARM] support arm target attribute, and warning for bad typo (PR #74812)

2023-12-07 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-backend-arm Author: None (hstk30-hw) Changes This adds support under ARM for the target("..") attributes like AArch64 https://reviews.llvm.org/D133848 . And warning for bad typo for "arch=". --- Full diff:

[clang] [Clang][ARM] support arm target attribute, and warning for bad typo (PR #74812)

2023-12-07 Thread via cfe-commits
https://github.com/hstk30-hw created https://github.com/llvm/llvm-project/pull/74812 This adds support under ARM for the target("..") attributes like AArch64 https://reviews.llvm.org/D133848 . And warning for bad typo for "arch=". >From 8a84eaf11bb76aba7db5243390f246d40b3f9630 Mon Sep 17