[PATCH] D105409: [docs] Fix linking issues in LibASTMatchers tutorial

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. jubnzv marked an inline comment as done. Closed by commit rG3697f2683695: [docs] Fix linking issues in LibASTMatchers tutorial (authored by jubnzv). Herald added a proj

[PATCH] D104097: [analyzer] Fix calculating offset for fields with an empty type

This revision was automatically updated to reflect the committed changes. Closed by commit rGc558b1fca735: [analyzer] Fix calculating offset for fields with an empty type (authored by jubnzv). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104097/new/

[PATCH] D104097: [analyzer] Fix calculating offset for fields with an empty type

jubnzv updated this revision to Diff 352361. jubnzv added a comment. Added two tests that check whether we still calculate padding correctly for structs with `[[no_unique_address]]` on two consecutive fields CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104097/new/ https://reviews.llvm

[PATCH] D104097: [analyzer] Fix calculating offset for fields with an empty type

jubnzv updated this revision to Diff 351423. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104097/new/ https://reviews.llvm.org/D104097 Files: clang/lib/StaticAnalyzer/Checkers/PaddingChecker.cpp clang/test/Analysis/padding_no_unique_address.cpp Index: clang/test/Analysis/padding_n

[PATCH] D104097: [analyzer] Fix calculating offset for fields with an empty type

jubnzv created this revision. jubnzv added reviewers: steakhal, NoQ, vsavchenko, Szelethus, xazax.hun, balazske. jubnzv added a project: clang. Herald added subscribers: manas, ASDenysPetrov, martong, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, rnkovacs, szepet, baloghadamsoftware. jubnzv req

[PATCH] D102569: [clang-tidy] Fix altera-struct-pack-align crash for struct fields with incomplete type

This revision was automatically updated to reflect the committed changes. Closed by commit rGab92a4c26f54: [clang-tidy] Fix altera-struct-pack-align crash for struct fields with… (authored by jubnzv). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D102

[PATCH] D102569: [clang-tidy] Fix altera-struct-pack-align crash for struct fields with incomplete type

jubnzv created this revision. jubnzv added reviewers: aaron.ballman, njames93. jubnzv added a project: clang-tools-extra. Herald added a subscriber: xazax.hun. jubnzv requested review of this revision. We can only use `ASTContext::getTypeInfo` for complete types. This fixes the following bugzilla

[PATCH] D102337: [clang-tidy] Fix test that requires Windows platofrm

This revision was automatically updated to reflect the committed changes. Closed by commit rGe07753c8814d: [clang-tidy] Fix test that requires Windows platofrm (authored by jubnzv). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D102337/new/ https://r

[PATCH] D102322: [clang-tidy] Add '-target' CLI option to override target triple

jubnzv abandoned this revision. jubnzv added a comment. Thank you, I didn't know that it is possible to set the target triple using `--extra-arg`. I created a new revision with this fix: https://reviews.llvm.org/D102337. For some reasons, the `--target=unknown-windows` doesn't work for me with t

[PATCH] D102337: [clang-tidy] Fix test that requires Windows platofrm

jubnzv created this revision. jubnzv added reviewers: aaron.ballman, njames93. jubnzv added a project: clang-tools-extra. Herald added subscribers: kbarton, xazax.hun, nemanjai. jubnzv requested review of this revision. This commit fixes the `cppcoreguidelines-pro-type-vararg` test when it runs o

[PATCH] D102322: [clang-tidy] Add '-target' CLI option to override target triple

jubnzv updated this revision to Diff 344758. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D102322/new/ https://reviews.llvm.org/D102322 Files: clang-tools-extra/clang-tidy/ClangTidy.cpp clang-tools-extra/clang-tidy/ClangTidy.h clang-tools-extra/clang-tidy/tool/ClangTidyMain.cpp

[PATCH] D102322: [clang-tidy] Add '-target' CLI option to override target triple

jubnzv created this revision. jubnzv added a project: clang-tools-extra. Herald added subscribers: kbarton, xazax.hun, nemanjai. jubnzv requested review of this revision. The added option allows the user to specify the target for which //clang-tidy// checks will be executed. This is necessary for

[PATCH] D101259: [clang-tidy] Fix cppcoreguidelines-pro-type-vararg false positives with __builtin_ms_va_list

jubnzv added inline comments. Comment at: clang-tools-extra/test/clang-tidy/checkers/cppcoreguidelines-pro-type-vararg-ms.cpp:7 + +// RUN: %check_clang_tidy %s cppcoreguidelines-pro-type-vararg %t + dyung wrote: > jubnzv wrote: > > probinson wrote: > > > probins

[PATCH] D101259: [clang-tidy] Fix cppcoreguidelines-pro-type-vararg false positives with __builtin_ms_va_list

jubnzv added inline comments. Comment at: clang-tools-extra/test/clang-tidy/checkers/cppcoreguidelines-pro-type-vararg-ms.cpp:7 + +// RUN: %check_clang_tidy %s cppcoreguidelines-pro-type-vararg %t + probinson wrote: > probinson wrote: > > TWeaver wrote: > > > nj

[PATCH] D101259: [clang-tidy] Fix cppcoreguidelines-pro-type-vararg false positives with __builtin_ms_va_list

This revision was automatically updated to reflect the committed changes. Closed by commit rGc2e9baf2e8da: [clang-tidy] Fix cppcoreguidelines-pro-type-vararg false positives with… (authored by jubnzv). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D10

[PATCH] D101259: [clang-tidy] Fix cppcoreguidelines-pro-type-vararg false positives with __builtin_ms_va_list

jubnzv updated this revision to Diff 342155. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101259/new/ https://reviews.llvm.org/D101259 Files: clang-tools-extra/clang-tidy/cppcoreguidelines/ProTypeVarargCheck.cpp clang-tools-extra/test/clang-tidy/checkers/cppcoreguidelines-pro-type-

[PATCH] D101259: [clang-tidy] Fix cppcoreguidelines-pro-type-vararg false positives with __builtin_ms_va_list

jubnzv updated this revision to Diff 341079. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101259/new/ https://reviews.llvm.org/D101259 Files: clang-tools-extra/clang-tidy/cppcoreguidelines/ProTypeVarargCheck.cpp clang-tools-extra/test/clang-tidy/checkers/cppcoreguidelines-pro-type-

[PATCH] D101259: [clang-tidy] Fix cppcoreguidelines-pro-type-vararg false positives with __builtin_ms_va_list

jubnzv added a comment. Thanks, I updated my patch and now CI is happy. I also found a few additional cases that can lead to false positives on the platforms which implements `__builtin_va_list` as `void *` or `char *`, and added additional checks for this. CHANGES SINCE LAST ACTION https://

[PATCH] D101259: [clang-tidy] Fix cppcoreguidelines-pro-type-vararg false positives with __builtin_ms_va_list

jubnzv updated this revision to Diff 340757. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101259/new/ https://reviews.llvm.org/D101259 Files: clang-tools-extra/clang-tidy/cppcoreguidelines/ProTypeVarargCheck.cpp clang-tools-extra/test/clang-tidy/checkers/cppcoreguidelines-pro-type-

[PATCH] D101259: [clang-tidy] Fix cppcoreguidelines-pro-type-vararg false positives with __builtin_ms_va_list

jubnzv updated this revision to Diff 340738. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101259/new/ https://reviews.llvm.org/D101259 Files: clang-tools-extra/clang-tidy/cppcoreguidelines/ProTypeVarargCheck.cpp clang-tools-extra/test/clang-tidy/checkers/cppcoreguidelines-pro-type-

[PATCH] D101259: [clang-tidy] Fix cppcoreguidelines-pro-type-vararg false positives with __builtin_ms_va_list

jubnzv updated this revision to Diff 340724. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101259/new/ https://reviews.llvm.org/D101259 Files: clang-tools-extra/clang-tidy/cppcoreguidelines/ProTypeVarargCheck.cpp clang-tools-extra/test/clang-tidy/checkers/cppcoreguidelines-pro-type-

[PATCH] D101259: [clang-tidy] Fix cppcoreguidelines-pro-type-vararg false positives with __builtin_ms_va_list

jubnzv created this revision. jubnzv added reviewers: njames93, aaron.ballman. jubnzv added a project: clang-tools-extra. Herald added subscribers: shchenz, kbarton, xazax.hun, nemanjai. jubnzv requested review of this revision. This commit fixes `cppcoreguidelines-pro-type-vararg` false positives

[PATCH] D99924: [clang-tidy] Avoid bugprone-macro-parentheses warnings after goto argument

This revision was automatically updated to reflect the committed changes. Closed by commit rG9a930aa5bd2f: [clang-tidy] Avoid bugprone-macro-parentheses warnings after goto argument (authored by jubnzv). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D