[PATCH] D149514: Check if First argument in _builtin_assume_aligned_ is of pointer type

2023-05-05 Thread Rishabh Bali via Phabricator via cfe-commits
Ris-Bali added a comment. Name : Rishabh Bali Github Username : Ris-Bali email-id : rishabhsb...@gmail.com CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149514/new/ https://reviews.llvm.org/D149514 ___ cfe-commits mailing list cfe-commits@lis

[PATCH] D149514: Check if First argument in _builtin_assume_aligned_ is of pointer type

2023-05-04 Thread Rishabh Bali via Phabricator via cfe-commits
Ris-Bali added a comment. I have made the requested changes. Please do let me know if other changes are required. Also I would like to thank everyone for their help and guidance. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149514/new/ https://reviews.llvm.org/D149514 ___

[PATCH] D149514: Check if First argument in _builtin_assume_aligned_ is of pointer type

2023-05-04 Thread Rishabh Bali via Phabricator via cfe-commits
Ris-Bali updated this revision to Diff 519628. Ris-Bali added a comment. Rebase CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149514/new/ https://reviews.llvm.org/D149514 Files: clang/docs/ReleaseNotes.rst clang/lib/CodeGen/CGBuiltin.cpp clang/lib/Sema/SemaChecking.cpp clang/te

[PATCH] D149514: Check if First argument in _builtin_assume_aligned_ is of pointer type

2023-05-04 Thread Rishabh Bali via Phabricator via cfe-commits
Ris-Bali updated this revision to Diff 519544. Ris-Bali marked 5 inline comments as done. Ris-Bali added a comment. Requested Changes CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149514/new/ https://reviews.llvm.org/D149514 Files: clang/docs/ReleaseNotes.rst clang/lib/CodeGen/CGBu

[PATCH] D149514: Check if First argument in _builtin_assume_aligned_ is of pointer type

2023-05-01 Thread Rishabh Bali via Phabricator via cfe-commits
Ris-Bali updated this revision to Diff 518543. Ris-Bali added a comment. Added test for function type CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149514/new/ https://reviews.llvm.org/D149514 Files: clang/docs/ReleaseNotes.rst clang/lib/Sema/SemaChecking.cpp clang/test/Sema/buil

[PATCH] D149514: Check if First argument in _builtin_assume_aligned_ is of pointer type

2023-05-01 Thread Rishabh Bali via Phabricator via cfe-commits
Ris-Bali marked an inline comment as done. Ris-Bali added a comment. I have made the requested changes. Please do let me know if any other changes are required CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149514/new/ https://reviews.llvm.org/D149514 __

[PATCH] D149514: Check if First argument in _builtin_assume_aligned_ is of pointer type

2023-05-01 Thread Rishabh Bali via Phabricator via cfe-commits
Ris-Bali updated this revision to Diff 518425. Ris-Bali marked 2 inline comments as done. Ris-Bali added a comment. Removed redundant code CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149514/new/ https://reviews.llvm.org/D149514 Files: clang/docs/ReleaseNotes.rst clang/lib/Sema/Se

[PATCH] D149514: Check if First argument in _builtin_assume_aligned_ is of pointer type

2023-04-30 Thread Rishabh Bali via Phabricator via cfe-commits
Ris-Bali updated this revision to Diff 518348. Ris-Bali added a comment. Removed redundant code. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149514/new/ https://reviews.llvm.org/D149514 Files: clang/docs/ReleaseNotes.rst clang/lib/Sema/SemaChecking.cpp clang/test/Sema/builtin-a

[PATCH] D149514: Check if First argument in _builtin_assume_aligned_ is of pointer type

2023-04-30 Thread Rishabh Bali via Phabricator via cfe-commits
Ris-Bali updated this revision to Diff 518300. Ris-Bali marked 4 inline comments as done. Ris-Bali added a comment. Array test added CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149514/new/ https://reviews.llvm.org/D149514 Files: clang/docs/ReleaseNotes.rst clang/lib/Sema/SemaChec

[PATCH] D149514: Check if First argument in _builtin_assume_aligned_ is of pointer type

2023-04-30 Thread Rishabh Bali via Phabricator via cfe-commits
Ris-Bali updated this revision to Diff 518293. Ris-Bali added a comment. Requested changes CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149514/new/ https://reviews.llvm.org/D149514 Files: clang/docs/ReleaseNotes.rst clang/lib/Sema/SemaChecking.cpp clang/test/Sema/builtin-assume-

[PATCH] D149514: Check if First argument in _builtin_assume_aligned_ is of pointer type

2023-04-29 Thread Rishabh Bali via Phabricator via cfe-commits
Ris-Bali updated this revision to Diff 518274. Ris-Bali added a comment. Herald added subscribers: carlosgalvezp, kbarton, nemanjai. Herald added a project: clang-tools-extra. Removed __builtin_assume_aligned from ignoredbuiltinstest() in clang-tools-extra/test/clang-tidy/checkers/cppcoreguidelin

[PATCH] D149514: Check if First argument in _builtin_assume_aligned_ is of pointer type

2023-04-29 Thread Rishabh Bali via Phabricator via cfe-commits
Ris-Bali updated this revision to Diff 518228. Ris-Bali added a comment. Added test and made changes in ReleaseNotes.rst CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149514/new/ https://reviews.llvm.org/D149514 Files: clang/docs/ReleaseNotes.rst clang/lib/Sema/SemaChecking.cpp c

[PATCH] D149514: Check if First argument in _builtin_assume_aligned_ is of pointer type

2023-04-29 Thread Rishabh Bali via Phabricator via cfe-commits
Ris-Bali updated this revision to Diff 518156. Ris-Bali added a comment. QualType error fix CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149514/new/ https://reviews.llvm.org/D149514 Files: clang/lib/Sema/SemaChecking.cpp Index: clang/lib/Sema/SemaChecking.cpp =

[PATCH] D149514: Check if First argument in _builtin_assume_aligned_ is of pointer type

2023-04-29 Thread Rishabh Bali via Phabricator via cfe-commits
Ris-Bali updated this revision to Diff 518153. Ris-Bali added a comment. Clang-format fix CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149514/new/ https://reviews.llvm.org/D149514 Files: clang/lib/Sema/SemaChecking.cpp Index: clang/lib/Sema/SemaChecking.cpp ===

[PATCH] D149514: Check if First argument in _builtin_assume_aligned_ is of pointer type

2023-04-29 Thread Rishabh Bali via Phabricator via cfe-commits
Ris-Bali created this revision. Ris-Bali added a reviewer: tbaeder. Ris-Bali added a project: clang. Herald added a project: All. Ris-Bali requested review of this revision. Herald added a subscriber: cfe-commits. Currently clang doesn't verify if the first argument in _builtin_assume_aligned_ is