rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
LGTM.
Repository:
rC Clang
https://reviews.llvm.org/D53714
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
rjmccall added a comment.
In https://reviews.llvm.org/D53738#1277172, @ebevhan wrote:
> I want to float the idea again of adding an AST type to encapsulate an
> arbitrary fixed-point semantic and using that as the 'common type' for binary
> operations involving fixed-point types. This would
rjmccall added inline comments.
Comment at: lib/AST/Expr.cpp:1609
case CK_AddressSpaceConversion:
-assert(getType()->isPointerType() || getType()->isBlockPointerType());
-assert(getSubExpr()->getType()->isPointerType() ||
-
rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
LGTM.
Comment at: include/clang/AST/Stmt.h:279
+/// in PredefinedExpr::IdentType.
+unsigned Type : 4;
+
rjmccall wrote:
> Since you're changing
rjmccall added inline comments.
Comment at: clang/lib/CodeGen/CGObjC.cpp:2480
+ SuppressResultRetain);
}
vsapsai wrote:
> rjmccall wrote:
> > This switch is just checking what you already computed as
> > `SuppressResultRetain`.
rjmccall added a comment.
This should at least be named `emitScalarConstant`.
https://reviews.llvm.org/D53725
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rjmccall added a comment.
I don't suppose there's any chance you can just tell Khronos to fix their
stuff. It's a little funny to be more conservative about keywords in C++ when
the C++ committee is actually much more aggressive about adding keywords in the
non-reserved space than C is.
Author: brad
Date: Fri Oct 26 18:14:22 2018
New Revision: 345443
URL: http://llvm.org/viewvc/llvm-project?rev=345443=rev
Log:
Revert "Pass the nopie flag to the linker when linking with -pg." until
one of the tests can be fixed on !OpenBSD hosts.
Modified:
neerajksingh updated this revision to Diff 171382.
neerajksingh added a comment.
Fix hans' comments.
https://reviews.llvm.org/D53457
Files:
docs/UsersManual.rst
include/clang/Driver/CLCompatOptions.td
include/clang/Driver/Driver.h
lib/Driver/Driver.cpp
test/Driver/cl-options.c
Author: brad
Date: Fri Oct 26 17:46:12 2018
New Revision: 345440
URL: http://llvm.org/viewvc/llvm-project?rev=345440=rev
Log:
Update the other test.
Modified:
cfe/trunk/test/Driver/openbsd.c
Modified: cfe/trunk/test/Driver/openbsd.c
URL:
rjmccall added a comment.
So, three points:
- That's not class-member-specific; you can have a placement `operator new[]`
at global scope that isn't the special `void*` placement operator and therefore
still has a cookie, and it would have just as much flexibility as a
class-member override
phosek created this revision.
phosek added reviewers: rsmith, rnk.
Herald added a subscriber: cfe-commits.
When the global new and delete operators aren't declared, Clang
provides and implicit declaration, but this declaration currently
always uses the default visibility. This is a problem when
Author: brad
Date: Fri Oct 26 17:23:28 2018
New Revision: 345439
URL: http://llvm.org/viewvc/llvm-project?rev=345439=rev
Log:
Pass the nopie flag to the linker when linking with -pg.
Modified:
cfe/trunk/lib/Driver/ToolChains/OpenBSD.cpp
cfe/trunk/test/Driver/openbsd.c
Modified:
Hello everyone,
Below are some buildbot numbers for the week of 10/7/018 - 10/13/2018.
Please see the same data in attached csv files:
The longest time each builder was red during the week;
"Status change ratio" by active builder (percent of builds that changed the
builder status from greed to
Hello everyone,
Below are some buildbot numbers for the last week of 10/14/2018 -
10/20/2018.
Please see the same data in attached csv files:
The longest time each builder was red during the week;
"Status change ratio" by active builder (percent of builds that changed the
builder status from
neerajksingh added a comment.
In https://reviews.llvm.org/D53457#1277315, @hans wrote:
> One note about flag ordering: the /clang: flags are concatenated to the end
> of
> the argument list, so in cases where the last flag wins, the /clang: flags
> will be chosen regardless of their order
rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
LGTM.
Repository:
rC Clang
https://reviews.llvm.org/D53715
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
Author: leonardchan
Date: Fri Oct 26 15:51:51 2018
New Revision: 345433
URL: http://llvm.org/viewvc/llvm-project?rev=345433=rev
Log:
Revert "[PassManager/Sanitizer] Enable usage of ported AddressSanitizer passes
with -fsanitize=address"
This reverts commit
david-salinas updated this revision to Diff 171367.
david-salinas added a comment.
remove __attribute__((cpu)) __attribute__((hc)) from test
Repository:
rC Clang
https://reviews.llvm.org/D53780
Files:
lib/CodeGen/CGCall.cpp
test/CodeGenCXX/address-space-cast-coerce.cpp
Index:
a_sidorin added a comment.
Hi Gabor,
I wonder if it is possible to get into situation where non-equivalent decls are
marked equivalent with this patch? If yes, we can create a mapping between
decls being imported and original decls as an alternative solution. However, I
cannot find any
jranieri-grammatech created this revision.
jranieri-grammatech added reviewers: aaron.ballman, klimek, sbenza.
Herald added a subscriber: cfe-commits.
This adds a matcher, isStaticLocal, that matches local static variables (i.e.
VarDecl::isStaticLocal).
Repository:
rC Clang
Author: vsapsai
Date: Fri Oct 26 15:16:24 2018
New Revision: 345432
URL: http://llvm.org/viewvc/llvm-project?rev=345432=rev
Log:
[VFS] Add property 'fallthrough' that controls fallback to real file system.
Default property value 'true' preserves current behavior. Value 'false' can be
used to
This revision was automatically updated to reflect the committed changes.
Closed by commit rL345431: [VFS] Add property fallthrough that
controls fallback to real file system. (authored by vsapsai, committed by ).
Herald added subscribers: llvm-commits, kristina.
Changed prior to commit:
Eugene.Zelenko added inline comments.
Comment at: docs/ReleaseNotes.rst:142
+
+ Diagnoses local variable declarations declaring more than one variable and
+ tries to refactor the code to one statement per declaration.
May be Finds or Detects like other checks?
david-salinas created this revision.
david-salinas added reviewers: yaxunl, scchan.
Herald added a subscriber: cfe-commits.
Coerced load/stores through memory do not take into account potential address
space differences when it creates its bitcasts.
Repository:
rC Clang
JonasToth updated this revision to Diff 171361.
JonasToth added a comment.
- [Fix] wrong condition in matcher coming from incorrect code change
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D51949
Files:
clang-tidy/readability/CMakeLists.txt
gtbercea updated this revision to Diff 171352.
gtbercea added a comment.
Add test.
Repository:
rC Clang
https://reviews.llvm.org/D53443
Files:
lib/CodeGen/CGOpenMPRuntimeNVPTX.cpp
test/OpenMP/nvptx_parallel_for_codegen.cpp
Index: test/OpenMP/nvptx_parallel_for_codegen.cpp
arphaman accepted this revision.
arphaman added a comment.
LGTM too when Aaron's comments are addressed
https://reviews.llvm.org/D53621
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
JonasToth updated this revision to Diff 171346.
JonasToth marked 16 inline comments as done.
JonasToth added a comment.
- address review comments
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D51949
Files:
clang-tidy/readability/CMakeLists.txt
JonasToth added inline comments.
Comment at: clang-tidy/readability/IsolateDeclarationCheck.cpp:52
+ const LangOptions ) {
+ assert(Indirections >= 0 && "Indirections must be non-negative");
+ if (Indirections == 0)
CaseyCarter updated this revision to Diff 171344.
CaseyCarter added a comment.
Clarify use of `numAllocs`.
https://reviews.llvm.org/D50860
Files:
test/std/thread/thread.threads/thread.thread.class/thread.thread.constr/F.pass.cpp
Index:
rsmith added inline comments.
Comment at: include/clang/AST/Stmt.h:1863
+ Expr *getCond() {
+return reinterpret_cast(getTrailingObjects()[condOffset()]);
+ }
riccibruno wrote:
> rsmith wrote:
> > Please `static_cast` rather than `reinterpret_cast`
JonasToth added inline comments.
Comment at: test/clang-tidy/misc-avoid-c-arrays.cpp:14
+
+template
+class array {
lebedev.ri wrote:
> JonasToth wrote:
> > Please add a case with templates, where `T` itself is the array type, maybe
> > there are other fancy
riccibruno updated this revision to Diff 171340.
riccibruno marked 6 inline comments as done.
riccibruno added a comment.
Add a flag in the output of -ast-dump indicating which sub-statement
the `ForStmt` is storing. This removes the ambiguity in the output
of -ast-dump.
Repository:
rC Clang
riccibruno updated this revision to Diff 171338.
riccibruno added a reviewer: rsmith.
riccibruno added a comment.
Add a flag to the output of -ast-dump indicating which sub-statement
a `WhileStmt` is storing.
Repository:
rC Clang
https://reviews.llvm.org/D53715
Files:
riccibruno updated this revision to Diff 171337.
riccibruno added a reviewer: rsmith.
riccibruno added a comment.
Add a flag to the output of -ast-dump indicating which sub-statement
a `SwitchStmt` is storing. This removes the ambiguity in the output
of -ast-dump as per rsmith's comment in
mstorsjo accepted this revision.
mstorsjo added a comment.
This revision is now accepted and ready to land.
Looks good to me, once the dependency patch is done.
Repository:
rC Clang
https://reviews.llvm.org/D53684
___
cfe-commits mailing list
riccibruno updated this revision to Diff 171336.
riccibruno added a reviewer: rsmith.
riccibruno added a comment.
Add a flag in the output of -ast-dump to indicate that
a `CaseStmt` is a GNU range case statement, following
rsmith's comment in https://reviews.llvm.org/D53717
Repository:
rC
Author: rsmith
Date: Fri Oct 26 12:42:43 2018
New Revision: 345423
URL: http://llvm.org/viewvc/llvm-project?rev=345423=rev
Log:
Fix test expectation to match reality.
Modified:
cfe/trunk/test/SemaCXX/alignof.cpp
Modified: cfe/trunk/test/SemaCXX/alignof.cpp
URL:
Author: rsmith
Date: Fri Oct 26 12:35:39 2018
New Revision: 345421
URL: http://llvm.org/viewvc/llvm-project?rev=345421=rev
Log:
Fix typo.
Modified:
cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td
Modified: cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td
URL:
rsmith added a comment.
This needs a test.
Comment at: lib/Driver/ToolChains/Linux.cpp:927-928
LibDir.str() + "/../include/c++",
+ // Some gcc installations put headers under "g++" without a
+ // version suffix.
+ LibDir.str() + "/../include/g++",
This revision was automatically updated to reflect the committed changes.
Closed by commit rL345419: PR26547: alignof should return ABI alignment, not
preferred alignment (authored by rsmith, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
Author: rsmith
Date: Fri Oct 26 12:26:45 2018
New Revision: 345419
URL: http://llvm.org/viewvc/llvm-project?rev=345419=rev
Log:
PR26547: alignof should return ABI alignment, not preferred alignment
Summary:
- Add `UETT_PreferredAlignOf` to account for the difference between `__alignof`
and
riccibruno updated this revision to Diff 171332.
riccibruno added a reviewer: rsmith.
riccibruno added a comment.
Add a flag to the output of -ast-dump indicating
which sub-statement `IfStmt` is storing. This removes
the ambiguity in the output of -ast-dump and addresses
rsmith's comment in
riccibruno updated this revision to Diff 171326.
riccibruno marked 4 inline comments as done.
riccibruno retitled this revision from "[AST] Pack PredefinedExpr" to "[AST]
Refactor PredefinedExpr".
riccibruno edited the summary of this revision.
riccibruno added a comment.
Address rjmcall's
lebedev.ri added inline comments.
Comment at: test/clang-tidy/misc-avoid-c-arrays.cpp:14
+
+template
+class array {
JonasToth wrote:
> Please add a case with templates, where `T` itself is the array type, maybe
> there are other fancy template tricks that
This revision was automatically updated to reflect the committed changes.
Closed by commit rCTE345418: [clang-doc] Switch to default to all-TUs executor
(authored by juliehockett, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D53170?vs=169414=171331#toc
Repository:
rCTE
Author: juliehockett
Date: Fri Oct 26 12:11:34 2018
New Revision: 345418
URL: http://llvm.org/viewvc/llvm-project?rev=345418=rev
Log:
[clang-doc] Switch to default to all-TUs executor
Since we generally want to document a whole project, not just one file.
Differential Revision:
Eugene.Zelenko added a comment.
May be this check belongs to modernize?
Comment at: docs/clang-tidy/checks/misc-avoid-c-arrays.rst:6
+
+Find C-style array delarations and recommend to use ``std::array<>``.
+All types of C arrays are diagnosed.
Finds.
aaron.ballman added inline comments.
Comment at: clang-tidy/readability/ConstValueReturnCheck.cpp:107
+diag(Def->getInnerLocStart(), "return type %0 is 'const'-qualified "
+ "hindering compiler optimizations")
+<<
This revision was automatically updated to reflect the committed changes.
Closed by commit rC345417: [NFC][OpenMP] Add new test for parallel for code
generation. (authored by gbercea, committed by ).
Repository:
rC Clang
https://reviews.llvm.org/D53772
Files:
Author: gbercea
Date: Fri Oct 26 11:59:52 2018
New Revision: 345417
URL: http://llvm.org/viewvc/llvm-project?rev=345417=rev
Log:
[NFC][OpenMP] Add new test for parallel for code generation.
Summary:
This is a simple test of the parallel for code generation. It will be used to
showcase the
ABataev accepted this revision.
ABataev added a comment.
This revision is now accepted and ready to land.
LG
Repository:
rC Clang
https://reviews.llvm.org/D53772
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
gtbercea created this revision.
gtbercea added reviewers: ABataev, caomhin.
Herald added subscribers: cfe-commits, guansong.
This is a simple test of the parallel for code generation. It will be used to
showcase the change introduced by patch https://reviews.llvm.org/D53443.
Repository:
rC
NoQ added inline comments.
Comment at: lib/StaticAnalyzer/Checkers/PaddingChecker.cpp:78-81
+// We need to be looking at a definition, not just any pointer to the
+// declaration.
+if (!(RD = RD->getDefinition()))
+ return;
tekknolagi wrote:
>
tekknolagi added inline comments.
Comment at: lib/StaticAnalyzer/Checkers/PaddingChecker.cpp:78-81
+// We need to be looking at a definition, not just any pointer to the
+// declaration.
+if (!(RD = RD->getDefinition()))
+ return;
tekknolagi
Anastasia added inline comments.
Comment at: test/SemaOpenCL/extension-begin.cl:43
g(0); // expected-error {{no matching function for call to 'g'}}
-// expected-note@-26 {{candidate unavailable as it requires OpenCL
extension 'my_ext' to be disabled}}
-//
CaseyCarter added a comment.
I'll put this explanation in the comments and push a change.
Comment at:
test/std/thread/thread.threads/thread.thread.class/thread.thread.constr/F.pass.cpp:121
//allocations.
+int numAllocs;
+
EricWF wrote:
> Why?
`main`
JonasToth added inline comments.
Comment at: docs/clang-tidy/checks/list.rst:13
abseil-redundant-strcat-calls
- abseil-string-find-startswith
abseil-str-cat-append
spurious change
Comment at: docs/clang-tidy/checks/list.rst:154
JonasToth added a comment.
`modernize` would be a fitting module, wouldn't it?
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D53771
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
hubert.reinterpretcast added inline comments.
Comment at: clang/lib/Sema/SemaOverload.cpp:3913
+for (auto Type : Types) {
+ if (S.Context.getCanonicalType(Type)->getTypeClass() != Type::Vector)
+return false;
hubert.reinterpretcast
ymandel marked 2 inline comments as done.
ymandel added inline comments.
Comment at: clang-tidy/readability/ConstValueReturnCheck.cpp:107
+diag(Def->getInnerLocStart(), "return type %0 is 'const'-qualified "
+ "hindering compiler
lebedev.ri created this revision.
lebedev.ri added reviewers: aaron.ballman, JonasToth, alexfh, hokein, xazax.hun.
lebedev.ri added a project: clang-tools-extra.
Herald added subscribers: rnkovacs, mgorny.
lebedev.ri edited the summary of this revision.
PR39224
greened created this revision.
greened added reviewers: danalbert, dlj, atanasyan, phosek.
Herald added a subscriber: cfe-commits.
Some gcc installations put C++ headers in PREFIX/include/g++ without indicating
a gcc version at all. Typically this is because the version is encoded
somewhere in
hubert.reinterpretcast added inline comments.
Comment at: clang/lib/Sema/SemaOverload.cpp:3913
+for (auto Type : Types) {
+ if (S.Context.getCanonicalType(Type)->getTypeClass() != Type::Vector)
+return false;
wuzish wrote:
>
scott.linder created this revision.
scott.linder added a reviewer: rsmith.
Herald added a subscriber: cfe-commits.
An attempt at removing one instance of a hardcoded output stream in
`CompilerInstance::ExecuteAction`. There are still other cases of output being
hard-coded to standard streams in
AlexeySachkov added inline comments.
Comment at: test/SemaOpenCL/extension-begin.cl:43
g(0); // expected-error {{no matching function for call to 'g'}}
-// expected-note@-26 {{candidate unavailable as it requires OpenCL
extension 'my_ext' to be disabled}}
-//
oleg.smolsky added a comment.
Folks, are there any other comments/suggestions?
Repository:
rC Clang
https://reviews.llvm.org/D52676
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
This revision was automatically updated to reflect the committed changes.
Closed by commit rC345398: [Fixed Point Arithmetic] Refactor fixed point casts
(authored by bjope, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D53707?vs=171113=171311#toc
Repository:
rC Clang
Author: bjope
Date: Fri Oct 26 09:12:12 2018
New Revision: 345398
URL: http://llvm.org/viewvc/llvm-project?rev=345398=rev
Log:
[Fixed Point Arithmetic] Refactor fixed point casts
Summary:
- Added names for some emitted values (such as "tobool" for
the result of a cast to boolean).
- Replaced
bjope added inline comments.
Comment at: clang/lib/CodeGen/CGExprScalar.cpp:3208
+ if (LHSWidth < CommonWidth)
+LHS = LHSSign ? Builder.CreateSExt(LHS, CommonTy)
+ : Builder.CreateZExt(LHS, CommonTy);
`LHS = Builder.CreateIntCast(LHS,
a_sidorin added a comment.
Hi Gabor,
I have tried to rebase it last week but there were some test failures. I hope
to fix them this weekend and finally commit the patch.
Repository:
rC Clang
https://reviews.llvm.org/D44100
___
cfe-commits
ABataev added inline comments.
Comment at: test/OpenMP/nvptx_parallel_for_codegen.cpp:44
+// CHECK: call void @__kmpc_kernel_prepare_parallel(
+// CHECK: call void @__kmpc_begin_sharing_variables(i8*** %shared_arg_refs,
i64 2)
+// CHECK: call void @llvm.nvvm.barrier0()
a_sidorin added a comment.
Hi Gabor,
Thank you for the patch. I looks reasonable but I have some questions.
Comment at: lib/AST/ASTImporter.cpp:2310
+D->getUnderlyingType(), FoundTypedef->getUnderlyingType())) {
+ QualType FromUT =
gtbercea updated this revision to Diff 171310.
gtbercea added a comment.
Add test.
Repository:
rC Clang
https://reviews.llvm.org/D53443
Files:
lib/CodeGen/CGOpenMPRuntimeNVPTX.cpp
test/OpenMP/nvptx_parallel_for_codegen.cpp
Index: test/OpenMP/nvptx_parallel_for_codegen.cpp
asavonic added inline comments.
Comment at: lib/Sema/SemaExpr.cpp:2247
if (R.empty() && HasTrailingLParen && II && !getLangOpts().CPlusPlus) {
-NamedDecl *D = ImplicitlyDefineFunction(NameLoc, *II, S);
-if (D) R.addDecl(D);
+if (getLangOpts().OpenCL) {
+ auto
martong added a comment.
Ping. This is still not committed, however, essential for ctu analyse any C++
project. Aleksei, I am happy to commit it for you if you don't have time, just
let me know.
Repository:
rC Clang
https://reviews.llvm.org/D44100
asavonic added a comment.
> TODO
> Measure the performance / size impact
I looks like the total size of OpenCLBuiltinDecl table should be
~450KB (for ~15000 functions). I guess it can be reduced furthermore by:
1. Replacing return type with an index (you've mentioned this in TODO).
2. Replace
ABataev accepted this revision.
ABataev added a comment.
This revision is now accepted and ready to land.
LG, with a nit
Comment at: lib/Sema/SemaOpenMP.cpp:5308
+
+// Build IV <= PrevEUB to be used in parallel for is in combination with
+// a distribute directive with
Anastasia accepted this revision.
Anastasia added a comment.
This revision is now accepted and ready to land.
LGTM! Thanks!
Comment at: test/SemaOpenCL/intel-subgroup-avc-ext-types.cl:5
+
+// All intel_sub_group_avc_* types can only be used as argument or return value
+// of
Anastasia added inline comments.
Comment at: lib/AST/Expr.cpp:1609
case CK_AddressSpaceConversion:
-assert(getType()->isPointerType() || getType()->isBlockPointerType());
-assert(getSubExpr()->getType()->isPointerType() ||
-
Anastasia created this revision.
Anastasia added reviewers: rjmccall, yaxunl.
I first enabled AS deduction for references that allowed to inherit the right
conversion diagnostics based on qualification conversion rules implemented
earlier for the pointer type.
Then in order to tests the
CaseyCarter created this revision.
CaseyCarter added reviewers: EricWF, mclow.lists.
Fairly straightforward: these tests were written without an implementation of
`<=>`, and they're incorrectly testing that `0 <=> foo` has the behavior that
is required for `foo <=> 0`.
aaron.ballman added inline comments.
Comment at: clang-tidy/readability/ConstValueReturnCheck.cpp:107
+diag(Def->getInnerLocStart(), "return type %0 is 'const'-qualified "
+ "hindering compiler optimizations")
+<<
Szelethus planned changes to this revision.
Szelethus added a comment.
@xazax.hun observed that the way `diagnostics` looks like is this:
diagnostics
report 1
notes
macro_expansions
path
executed_lines
report 2
...
Bt, if I didn't insist on this
Anastasia accepted this revision.
Anastasia added a comment.
This revision is now accepted and ready to land.
LGTM! Thanks!
Comment at: test/SemaOpenCL/extension-begin.cl:43
g(0); // expected-error {{no matching function for call to 'g'}}
-// expected-note@-26
aaron.ballman added a comment.
Mostly minor nits that clean up wording and comments.
Comment at: clang-tidy/readability/IsolateDeclarationCheck.cpp:52
+ const LangOptions ) {
+ assert(Indirections >= 0 && "Indirections must be
lebedev.ri added inline comments.
Comment at: test/clang-tidy/readability-isolate-declaration.cpp:233
+ int member1, member2;
+ // TODO: Transform FieldDecl's as well
+};
Comment is misleading. Transform == fixit, at least for me.
But they are not even
hans added a comment.
One note about flag ordering: the /clang: flags are concatenated to the end of
the argument list, so in cases where the last flag wins, the /clang: flags
will be chosen regardless of their order relative to other flags on the driver
command line.
This seems a little
ymandel added inline comments.
Comment at: clang-tidy/readability/ConstValueReturnCheck.cpp:107
+diag(Def->getInnerLocStart(), "return type %0 is 'const'-qualified "
+ "hindering compiler optimizations")
+<<
ymandel updated this revision to Diff 171293.
ymandel marked 4 inline comments as done.
ymandel added a comment.
Reword comment on the check, for clarity.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D53025
Files:
clang-tidy/readability/CMakeLists.txt
riccibruno added a comment.
In https://reviews.llvm.org/D53717#1276388, @rsmith wrote:
> In https://reviews.llvm.org/D53717#1276245, @rsmith wrote:
>
> > Do you have evidence that this complexity is worthwhile? (I wouldn't
> > imagine we have very many `ForStmt`s per translation unit, so saving
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM aside from some small nits.
Comment at: clang/docs/LanguageExtensions.rst:2666
- #pragma clang attribute push(__attribute__((annotate("custom"))),
balazske updated this revision to Diff 171292.
balazske added a comment.
- Added missing Import_New with Selector and DeclarationName.
Repository:
rC Clang
https://reviews.llvm.org/D53751
Files:
include/clang/AST/ASTImporter.h
lib/AST/ASTImporter.cpp
Index: lib/AST/ASTImporter.cpp
balazske added inline comments.
Comment at: lib/AST/ASTImporter.cpp:7716
}
}
This can be simplified by removing brace characters and removing `ToD`.
Repository:
rC Clang
https://reviews.llvm.org/D53755
This revision was automatically updated to reflect the committed changes.
Closed by commit rL345381: [clang-tidy] Re-commit: Add new
readability-uppercase-literal-suffix check… (authored by lebedevri,
committed by ).
Changed prior to commit:
aaron.ballman added inline comments.
Comment at: clang-tidy/readability/ConstValueReturnCheck.cpp:107
+diag(Def->getInnerLocStart(), "return type %0 is 'const'-qualified "
+ "hindering compiler optimizations")
+<<
Author: lebedevri
Date: Fri Oct 26 06:09:27 2018
New Revision: 345381
URL: http://llvm.org/viewvc/llvm-project?rev=345381=rev
Log:
[clang-tidy] Re-commit: Add new 'readability-uppercase-literal-suffix' check
(CERT DCL16-C, MISRA C:2012, 7.3, MISRA C++:2008, 2-13-4)
Summary:
Detects when the
It seems the generated MS mangling differs between 32- and 64-bit, and
the test expects the latter.
Let's revert in the meantime.. r345380
On Fri, Oct 26, 2018 at 1:10 PM, Peter Smith via cfe-commits
wrote:
> This commit, specifically the changes made to
>
lebedev.ri added a comment.
Rerun on clang-tools-extra+lld+clang codebase, nothing broken.
I'm gonna reland.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D52670
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
1 - 100 of 124 matches
Mail list logo