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
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
___
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
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
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
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
__
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
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
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
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-
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
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
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
=
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
===
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
15 matches
Mail list logo