This revision was automatically updated to reflect the committed changes.
Closed by commit rG1c70dec18c7e: [libunwind] Remove __FILE__ and __LINE__ from
error reporting (authored by leonardchan).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75890/ne
tetsuo-cpp marked an inline comment as done.
tetsuo-cpp added a comment.
@njames93 @MaskRay
Thanks for helping me with testing. I'll remember this for next time.
I also saw this pattern of retrieving `LangOptions` from an `ASTContext` in
some other checks (`RedundantExpressionCheck` and `Static
sameerds added a comment.
The commit summary needs improvement. The syntax is not really necessary there,
but instead this needs a better explanation of how this builtin fits in with
the overall scheme of language-specific and target-specific details of an
atomic operation. For example, is this
PaulkaToast updated this revision to Diff 249544.
PaulkaToast added a comment.
Addressed @Eugene.Zelenko comments.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75332/new/
https://reviews.llvm.org/D75332
Files:
clang-tools-extra/clang-tidy/CMake
shiva0217 updated this revision to Diff 249545.
shiva0217 added a comment.
Update patch to address @evandro's comments.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57497/new/
https://reviews.llvm.org/D57497
Files:
clang/docs/ClangCommandLineRe
skan updated this revision to Diff 249546.
skan added a comment.
Address review comments
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75723/new/
https://reviews.llvm.org/D75723
Files:
clang/lib/Headers/ia32intrin.h
clang/test/CodeGen/bitscan-
skan updated this revision to Diff 249547.
skan added a comment.
Address review comment
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75894/new/
https://reviews.llvm.org/D75894
Files:
clang/lib/Headers/bmiintrin.h
clang/test/CodeGen/bmi-builti
craig.topper added inline comments.
Comment at: clang/lib/Headers/ia32intrin.h:456
+ __extension__({
\
+long long c = (long long)(b);
\
+unsigned char d;
skan updated this revision to Diff 249548.
skan added a comment.
Fix typo
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75896/new/
https://reviews.llvm.org/D75896
Files:
clang/lib/Headers/cldemoteintrin.h
clang/test/CodeGen/cldemote.c
Index:
craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75894/new/
https://reviews.llvm.org/D75894
___
craig.topper added inline comments.
Comment at: clang/lib/Headers/ia32intrin.h:417
+#define _BitScanForward(a, b)
\
+ __extension__({
\
+int c = (int)(b);
jrtc27 added a comment.
I believe my previous comments have indeed been addressed.
Comment at: clang/docs/ClangCommandLineReference.rst:2958
+
+Put global and static data smaller than the limitation into a special section
(RISCV only)
+
lenary wrote:
> "(RISC-
oontvoo created this revision.
oontvoo added a reviewer: jyknight.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
remove redundant pred
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D75971
Files:
clang/include/clang/Lex/Preprocessor.h
Index: clang/i
craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75896/new/
https://reviews.llvm.org/D75896
___
craig.topper added inline comments.
Comment at: clang/lib/Headers/ia32intrin.h:417
+#define _BitScanForward(a, b)
\
+ __extension__({
\
+int c = (int)(b);
skan updated this revision to Diff 249550.
skan marked an inline comment as done.
skan added a comment.
Make the variale name in macro start with 2 underscores
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75723/new/
https://reviews.llvm.org/D75723
skan added inline comments.
Comment at: clang/lib/Headers/ia32intrin.h:456
+ __extension__({
\
+long long c = (long long)(b);
\
+unsigned char d;
Author: Akira Hatanaka
Date: 2020-03-10T22:10:32-07:00
New Revision: 37fa9d65eaa7d8fae7e5b787c4e8ef0a982cead5
URL:
https://github.com/llvm/llvm-project/commit/37fa9d65eaa7d8fae7e5b787c4e8ef0a982cead5
DIFF:
https://github.com/llvm/llvm-project/commit/37fa9d65eaa7d8fae7e5b787c4e8ef0a982cead5.diff
craig.topper added inline comments.
Comment at: clang/lib/Headers/ia32intrin.h:421
+if (__c != 0) {
\
+ *(a) = (unsigned)__bsfd(__c);
\
+ __d = 1;
shiva0217 updated this revision to Diff 249558.
shiva0217 added a comment.
Update patch to address @jrtc27's comment.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57497/new/
https://reviews.llvm.org/D57497
Files:
clang/docs/ClangCommandLineRefe
skan marked an inline comment as done.
skan added inline comments.
Comment at: clang/lib/Headers/ia32intrin.h:421
+if (__c != 0) {
\
+ *(a) = (unsigned)__bsfd(__c);
\
201 - 221 of 221 matches
Mail list logo