a.sidorin closed this revision.
a.sidorin added a comment.
Closed with https://reviews.llvm.org/rC332256.
Repository:
rC Clang
https://reviews.llvm.org/D46398
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/m
craig.topper created this revision.
craig.topper added reviewers: RKSimon, spatel.
I believe this is safe assuming default rounding mode. The conversion might be
inexact, but it can never overflow the FP type so this shouldn't be undefined
behavior for the uitofp/sitofp instructions.
We already
yvvan added inline comments.
Comment at: lib/Sema/SemaCodeComplete.cpp:4118
+ : diag::err_typecheck_member_reference_suggestion;
+ Diag(OpLoc, DiagID) << ConvertedBaseType << IsArrow
+ << Base->getSourceRange() << FixIt;
yvvan updated this revision to Diff 146749.
yvvan marked 2 inline comments as done.
yvvan added a comment.
Only C++ part, libclang part has moved to https://reviews.llvm.org/D46862
https://reviews.llvm.org/D41537
Files:
include/clang/Basic/DiagnosticSemaKinds.td
include/clang/Sema/CodeCompl
yvvan created this revision.
yvvan added reviewers: ilya-biryukov, klimek, bkramer, arphaman.
Follow up for https://reviews.llvm.org/D41537 - libclang part is extracted into
this review
https://reviews.llvm.org/D46862
Files:
include/clang-c/Index.h
test/Index/complete-arrow-dot.cpp
tools
xazax.hun added inline comments.
Comment at: include/clang/CrossTU/CrossTranslationUnit.h:117
- /// This function loads a function definition from an external AST
- ///file.
+ /// \brief This function loads a definition from an external AST file.
///
-
dexonsmith requested changes to this revision.
dexonsmith added a comment.
This revision now requires changes to proceed.
The code looks correct to me, although I have a few suggestions inline. When
you resubmit, please include full context (e.g., `git diff -U99`).
Jan and I discussed this
Author: jvesely
Date: Mon May 14 21:22:43 2018
New Revision: 332324
URL: http://llvm.org/viewvc/llvm-project?rev=332324&view=rev
Log:
rootn: Use denormal path only
It's OK to either flush to 0 or return denormal result if the device
does not support denormals. See sec 7.2 and 7.5.3 of OCL specs
U
Author: ctopper
Date: Mon May 14 20:17:52 2018
New Revision: 332322
URL: http://llvm.org/viewvc/llvm-project?rev=332322&view=rev
Log:
[X86] Revert part of r332266: Use __builtin_convertvector to replace some of
the avx512 truncate builtins.
The masking doesn't work right in the backend for the o
shiva0217 added inline comments.
Comment at: lib/Driver/ToolChains/Arch/RISCV.cpp:130
+ // which may override the defaults.
+ handleTargetFeaturesGroup(Args, Features,
options::OPT_m_riscv_Features_Group);
}
kito-cheng wrote:
> This part should move to the b
shiva0217 updated this revision to Diff 146737.
shiva0217 added a comment.
Rebase patch to https://reviews.llvm.org/D45465
Repository:
rL LLVM
https://reviews.llvm.org/D44888
Files:
include/clang/Driver/Options.td
lib/Driver/ToolChains/Arch/RISCV.cpp
test/Driver/riscv-features.c
Inde
Quuxplusone added a comment.
In https://reviews.llvm.org/D45766#1097797, @ksu.shadura wrote:
> Thank you for the test example! I got your point, but I wanted to ask if it
> should be like this for commutative operations?
> In our case it is actually matrix, and subtraction of matrices is not
>
kent08ian added a comment.
In https://reviews.llvm.org/D10833#970906, @milianw wrote:
> still looks good to me. can someone else please review and commit this?
Any updates on this? This feature will be very helpful. :)
Repository:
rC Clang
https://reviews.llvm.org/D10833
___
This revision was automatically updated to reflect the committed changes.
Closed by commit rL332314: [AST] Fix printing tag decl groups in decl contexts
(authored by jdenny, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D45465?vs=1467
Author: jdenny
Date: Mon May 14 17:44:14 2018
New Revision: 332314
URL: http://llvm.org/viewvc/llvm-project?rev=332314&view=rev
Log:
[AST] Fix printing tag decl groups in decl contexts
For example, given:
struct T1 {
struct T2 *p0;
};
-ast-print produced:
struct T1 {
struct T2;
bruno added a comment.
Ping
Repository:
rC Clang
https://reviews.llvm.org/D46485
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jdenny added a comment.
In https://reviews.llvm.org/D45465#1098710, @rsmith wrote:
> Looks good, thanks.
Thanks.
> It strikes me that this will still lead to inconsistencies. For example, I
> expect this:
>
> struct A { struct B *a, *b; struct B *c, *d; };
>
>
> ... to print as:
>
> s
efriedma added a comment.
Could you include some documentation for how to construct a baremetal
environment like the one this code expects? It's not clear what exactly you
expect to be installed where.
Repository:
rC Clang
https://reviews.llvm.org/D46822
NoQ added a comment.
Yay thanks!
I think some cornercases would need to be dealt with.
Comment at: lib/StaticAnalyzer/Core/RegionStore.cpp:1650
+
+// If there is a list, but no init, it must be zero.
+if (i >= InitList->getNumInits())
N
Author: rtrieu
Date: Mon May 14 16:21:48 2018
New Revision: 332310
URL: http://llvm.org/viewvc/llvm-project?rev=332310&view=rev
Log:
Enable control flow pruning of float overflow warnings.
Like other conversion warnings, allow float overflow warnings to be disabled
in known dead paths of template
rsmith added a comment.
This would violate our guidelines for diagnostic messages; see
https://clang.llvm.org/docs/InternalsManual.html#the-format-string
In particular: "Take advantage of location information. The user will be able
to see the line and location of the caret, so you don’t need to
pfultz2 updated this revision to Diff 146719.
pfultz2 added a comment.
Rebased
https://reviews.llvm.org/D46159
Files:
clang-tidy/ClangTidy.cpp
clang-tidy/ClangTidy.h
clang-tidy/ClangTidyDiagnosticConsumer.cpp
clang-tidy/ClangTidyDiagnosticConsumer.h
clang-tidy/tool/ClangTidyMain.cpp
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
Looks good, thanks.
It strikes me that this will still lead to inconsistencies. For example, I
expect this:
struct A { struct B *a, *b; struct B *c, *d; };
... to print as:
struct A {
vsapsai added a comment.
Thanks for the review.
Repository:
rL LLVM
https://reviews.llvm.org/D46446
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
This revision was automatically updated to reflect the committed changes.
Closed by commit rL332307: [c++17] Fix assertion on synthesizing deduction
guides after a fatal error. (authored by vsapsai, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://review
Author: vsapsai
Date: Mon May 14 15:49:44 2018
New Revision: 332307
URL: http://llvm.org/viewvc/llvm-project?rev=332307&view=rev
Log:
[c++17] Fix assertion on synthesizing deduction guides after a fatal error.
After a fatal error Sema::InstantiatingTemplate doesn't allow further
instantiation and
jdenny updated this revision to Diff 146709.
jdenny marked 2 inline comments as done.
jdenny added a comment.
Made the suggested change. Thanks!
https://reviews.llvm.org/D45465
Files:
lib/AST/DeclPrinter.cpp
test/Misc/ast-print-enum-decl.c
test/Misc/ast-print-record-decl.c
test/Sema/as
rsmith added a comment.
How do you deal with the fact that node handles for map-like types need to be
able to mutate a const member of a `pair` in-place? Are you assuming/hoping the
compiler won't do bad things to the program as a result, or is there some
reason why it would be unlikely to do s
This revision was automatically updated to reflect the committed changes.
Closed by commit rC332303: [analyzer] Re-apply r331096 "CStringChecker:
Add support for BSD strlcpy()...". (authored by dergachev, committed by ).
Repository:
rC Clang
https://reviews.llvm.org/D45177
Files:
lib/Static
Author: dergachev
Date: Mon May 14 15:32:24 2018
New Revision: 332303
URL: http://llvm.org/viewvc/llvm-project?rev=332303&view=rev
Log:
[analyzer] Re-apply r331096 "CStringChecker: Add support for BSD strlcpy()...".
Fixed after revert in r331401.
Patch by David Carlier!
Differential Revision: h
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
Thanks! I'll commit again.
https://reviews.llvm.org/D45177
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailma
NoQ added a comment.
Thank you! Looks good overall.
Comment at: lib/StaticAnalyzer/Checkers/CStringChecker.cpp:1013-1014
+
+ // Get the offset and the base region to figure out whether the offset of
+ // buffer is 0.
+ RegionOffset Offset = MR->getAsOffset();
---
yunlian updated this revision to Diff 146698.
yunlian edited the summary of this revision.
yunlian added a comment.
Herald added subscribers: JDevlieghere, aprantl.
Use the option -gsplit-dwarf to make it more consistent.
https://reviews.llvm.org/D44788
Files:
include/clang/Driver/Options.td
NoQ added a comment.
In https://reviews.llvm.org/D46187#1089087, @lebedev.ri wrote:
> > I believe we could also benefit from a method of extracting the analyzer's
> > `clang -cc1` run-line from clang-tidy. This would allow arbitrary debugging
> > over a single analyzer invocation.
>
> I'm not r
This revision was automatically updated to reflect the committed changes.
Closed by commit rC332300: [analyzer] Extend the ObjCAutoreleaseWriteChecker to
warn on captures as well (authored by george.karpenkov, committed by ).
Herald added a subscriber: cfe-commits.
Repository:
rC Clang
https:/
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
LGTM, thanks!
https://reviews.llvm.org/D46446
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listi
Author: george.karpenkov
Date: Mon May 14 14:39:54 2018
New Revision: 332300
URL: http://llvm.org/viewvc/llvm-project?rev=332300&view=rev
Log:
[analyzer] Extend the ObjCAutoreleaseWriteChecker to warn on captures as well
A common pattern is that the code in the block does not write into the
varia
rsmith added inline comments.
Comment at: lib/AST/DeclPrinter.cpp:393-397
if (!BaseType.isNull() && isa(BaseType))
BaseType = cast(BaseType)->getNamedType();
if (!BaseType.isNull() && isa(BaseType) &&
- cast(BaseType)->getDecl() == Decls[0]) {
+
rsmith added a comment.
Clang's current behavior is observably wrong in MS compatibility mode. For
example:
namespace N {
enum E *p;
exterm E e; // ok, finds E injected into scope by previous declaration
}
N::E e; // error, no E in N!
namespace N {
extern E e; // error, no E
vsapsai added a comment.
Ping.
https://reviews.llvm.org/D46446
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: jdenny
Date: Mon May 14 14:06:04 2018
New Revision: 332294
URL: http://llvm.org/viewvc/llvm-project?rev=332294&view=rev
Log:
Fix test fail on some buildbots, caused by r332281.
Modified:
cfe/trunk/test/Misc/ast-print-record-decl.c
Modified: cfe/trunk/test/Misc/ast-print-record-decl.c
Author: rsmith
Date: Mon May 14 13:52:48 2018
New Revision: 332291
URL: http://llvm.org/viewvc/llvm-project?rev=332291&view=rev
Log:
Fix regression in r332076.
If the name after 'template' is an unresolved using declaration (not containing
'typename'), then we don't yet know if it's a valid templ
zinovy.nis updated this revision to Diff 146677.
zinovy.nis added a comment.
- Fix for templated type names. Thanks AlexanderK for pointing.
https://reviews.llvm.org/D45927
Files:
clang-tidy/modernize/UseAutoCheck.cpp
docs/clang-tidy/checks/modernize-use-auto.rst
test/clang-tidy/modernize
ioeric updated this revision to Diff 146675.
ioeric marked 7 inline comments as done.
ioeric added a comment.
- Addressed review comments.
- Merged with origin/master
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D46497
Files:
clangd/ClangdServer.cpp
clangd/CodeComplete.cpp
jdenny created this revision.
jdenny added reviewers: aaron.ballman, rsmith, hfinkel.
For example, given:
enum __attribute__((deprecated)) T *p;
-ast-print produced:
enum T *p;
The trouble was that the EnumDecl node was suppressed, as revealed by
-ast-dump. The suppression of the EnumDecl
Author: george.karpenkov
Date: Mon May 14 13:29:16 2018
New Revision: 332288
URL: http://llvm.org/viewvc/llvm-project?rev=332288&view=rev
Log:
Remove the fixit for the diagnostics regarding capturing autoreleasing
variables in a block
The fixit is actively harmful, as it encourages developers to
Author: ioeric
Date: Mon May 14 13:17:53 2018
New Revision: 332287
URL: http://llvm.org/viewvc/llvm-project?rev=332287&view=rev
Log:
[Tooling] Pull #include manipulation code from clangFormat into libToolingCore.
Summary: Also pull #include related style out of FormatStyle as
tooling::IncludeSty
This revision was automatically updated to reflect the committed changes.
Closed by commit rL332287: [Tooling] Pull #include manipulation code from
clangFormat into libToolingCore. (authored by ioeric, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews
Author: rsmith
Date: Mon May 14 13:15:04 2018
New Revision: 332286
URL: http://llvm.org/viewvc/llvm-project?rev=332286&view=rev
Log:
PR37450: Fix bug that disabled some type checks for variables with deduced
types.
Also improve diagnostic for the case where a type is non-literal because it's a
jdenny updated this revision to Diff 146668.
jdenny added a comment.
Rebased. Ping.
https://reviews.llvm.org/D45465
Files:
lib/AST/DeclPrinter.cpp
test/Misc/ast-print-enum-decl.c
test/Misc/ast-print-record-decl.c
test/Sema/ast-print.c
test/SemaCXX/ast-print.cpp
Index: test/SemaCXX/a
ioeric updated this revision to Diff 14.
ioeric added a comment.
- Merged with origin/master
- Make copy instead of taking references to ensure lifetime.
Repository:
rC Clang
https://reviews.llvm.org/D46496
Files:
include/clang/Format/Format.h
include/clang/Tooling/Core/HeaderInclude
This revision was automatically updated to reflect the committed changes.
Closed by commit rL332284: [clang-format] Move #include related style to
libToolingCore (authored by ioeric, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews.llvm.org/D46758
F
Author: ioeric
Date: Mon May 14 12:51:33 2018
New Revision: 332284
URL: http://llvm.org/viewvc/llvm-project?rev=332284&view=rev
Log:
[clang-format] Move #include related style to libToolingCore
Summary: This will be shared by include insertion/deletion library.
Reviewers: ilya-biryukov
Reviewed
thakis accepted this revision.
thakis added inline comments.
This revision is now accepted and ready to land.
Comment at: test/Index/complete-and-plugins.c:3
+
+// RUN: c-index-test -code-completion-at=%s:6:1 -load
%llvmshlibdir/PrintFunctionNames%pluginext -add-plugin print-fns
Author: vsapsai
Date: Mon May 14 12:45:24 2018
New Revision: 332282
URL: http://llvm.org/viewvc/llvm-project?rev=332282&view=rev
Log:
Update XFAIL so apple-clang-9.0 is the last version not implementing Core 2094.
The test is passing with apple-clang-9.1. rdar://problem/40222003
Modified:
l
This revision was automatically updated to reflect the committed changes.
jdenny marked an inline comment as done.
Closed by commit rL332281: [AST] Print correct tag decl for tag specifier
(authored by jdenny, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
htt
This revision was automatically updated to reflect the committed changes.
Closed by commit rC332281: [AST] Print correct tag decl for tag specifier
(authored by jdenny, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D45463?vs=146351&id=146658#toc
Repository:
rL LLVM
https
Author: jdenny
Date: Mon May 14 12:36:45 2018
New Revision: 332281
URL: http://llvm.org/viewvc/llvm-project?rev=332281&view=rev
Log:
[AST] Print correct tag decl for tag specifier
For example, given:
void fn() {
struct T *p0;
struct T { int i; } *p1;
}
-ast-print produced:
void f
rsmith added a comment.
Also we probably want to hold off on landing this until PR37458 is fixed,
otherwise `std::launder` will start miscompiling code.
https://reviews.llvm.org/D40218
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://
This revision was automatically updated to reflect the committed changes.
Closed by commit rC332279: CodeGen: Emit string literal in constant address
space (authored by yaxunl, committed by ).
Repository:
rC Clang
https://reviews.llvm.org/D46643
Files:
lib/CodeGen/CGDecl.cpp
lib/CodeGen/C
Author: yaxunl
Date: Mon May 14 12:20:12 2018
New Revision: 332279
URL: http://llvm.org/viewvc/llvm-project?rev=332279&view=rev
Log:
CodeGen: Emit string literal in constant address space
Some targets have constant address space (e.g. amdgcn). For them string literal
should be
emitted in constan
Author: eugenezelenko
Date: Mon May 14 12:12:40 2018
New Revision: 332277
URL: http://llvm.org/viewvc/llvm-project?rev=332277&view=rev
Log:
[Documentation] Alphabetical order of new checks in Release Notes.
Modified:
clang-tools-extra/trunk/docs/ReleaseNotes.rst
Modified: clang-tools-extra/t
rsmith added a comment.
In https://reviews.llvm.org/D46112#1096893, @aaron.ballman wrote:
> In https://reviews.llvm.org/D46112#1091981, @efriedma wrote:
>
> > Also needs some test coverage for atomic operations which aren't calls,
> > like "typedef struct S S; void f(_Atomic S *s, _Atomic S *s2)
This revision was automatically updated to reflect the committed changes.
Closed by commit rC332275: [AST] Fix -ast-print for _Bool when have diagnostics
(authored by jdenny, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D45093?vs=145711&id=146650#toc
Repository:
rC Clang
rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
Thanks, LGTM.
https://reviews.llvm.org/D46643
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/l
Author: jdenny
Date: Mon May 14 11:41:44 2018
New Revision: 332275
URL: http://llvm.org/viewvc/llvm-project?rev=332275&view=rev
Log:
[AST] Fix -ast-print for _Bool when have diagnostics
For example, given:
#define bool _Bool
_Bool i;
void fn() { 1; }
-ast-print produced:
tmp.c:3:13: wa
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
LGTM with a fix (and test) for pointer-to-array-of-dynamic-class-type handling.
Comment at: lib/CodeGen/CGBuiltin.cpp:936-938
+ const auto *Record = Ty->getAsCXXRecordDecl()
vsk added a comment.
Sgtm. I think it would be worthwhile to release-note this.
Repository:
rC Clang
https://reviews.llvm.org/D46694
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
yaxunl updated this revision to Diff 146647.
yaxunl marked 2 inline comments as done.
yaxunl added a comment.
Revised by John's comments.
https://reviews.llvm.org/D46643
Files:
lib/CodeGen/CGDecl.cpp
lib/CodeGen/CodeGenModule.cpp
lib/CodeGen/CodeGenModule.h
test/CodeGenCXX/amdgcn-string
probinson added a comment.
In https://reviews.llvm.org/D46767#1096746, @rsmith wrote:
> Everything old is new again.
If only that were true about my brain. :-P
> This was discussed when `-fclang-abi-compat` was introduced; see
> https://reviews.llvm.org/D36501 for the argument why this patch
lebedev.ri added a project: clang-tools-extra.
lebedev.ri added a comment.
In https://reviews.llvm.org/D46602#1098092, @alexfh wrote:
> In https://reviews.llvm.org/D46602#1097954, @lebedev.ri wrote:
>
> > How do i reflect that in tests? The output name will basically be random.
>
>
> Why random?
This revision was automatically updated to reflect the committed changes.
Closed by commit rC332266: [X86] Use __builtin_convertvector to replace some of
the avx512 truncate… (authored by ctopper, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D46742?vs=146286&id=146645#toc
Author: ctopper
Date: Mon May 14 10:50:40 2018
New Revision: 332266
URL: http://llvm.org/viewvc/llvm-project?rev=332266&view=rev
Log:
[X86] Use __builtin_convertvector to replace some of the avx512 truncate
builtins.
As long as the destination type is a 256 or 128 bit vector with the same number
lebedev.ri updated this revision to Diff 146644.
lebedev.ri edited the summary of this revision.
lebedev.ri added subscribers: aaron.ballman, JonasToth.
lebedev.ri added a comment.
- Get rid of 'prefix elision'
- Don't use full source file name, only the filename
- Prefix that filename with ISO860
zahiraam updated this revision to Diff 146642.
https://reviews.llvm.org/D43576
Files:
include/clang/AST/Decl.h
include/clang/AST/RecursiveASTVisitor.h
include/clang/Basic/Attr.td
include/clang/Basic/DeclNodes.td
include/clang/Sema/AttributeList.h
include/clang/Sema/Sema.h
lib/AST/De
jakehehrlich added a comment.
I'm generally ok with this but I think some important checks need to occur
first. Namely I think I'd like to check that Chromium still builds with this
option since I'd imagine they'd be affected by this.
Repository:
rC Clang
https://reviews.llvm.org/D46791
jakehehrlich added inline comments.
Comment at: lib/Driver/ToolChains/Clang.cpp:4818
// Handle the debug info splitting at object creation time if we're
// creating an object.
if (SplitDWARF && Output.getType() == types::TY_Object)
Maybe re-add a TODO to
juliehockett added a comment.
Pingany more thoughts?
https://reviews.llvm.org/D43341
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mikhail.ramalho added a comment.
> Could you provide a minimal example where USRs are not generated? It might
> be the case that there are other ways to fix it.
Sure, I'll try to reduce our testcase, but basically we have an
ASTFrontendAction [0] that adds a set of intrinsics [1] to the preproce
rjmccall added inline comments.
Comment at: lib/CodeGen/CGDecl.cpp:1373
+ llvm::Type *BP = llvm::Type::getInt8Ty(CGM.getLLVMContext())
+ ->getPointerTo(Loc.getAddressSpace());
if (Loc.getType() != BP)
`CGM.Int8Ty` exists.
=
Sunil_Srivastava created this revision.
Herald added a subscriber: cfe-commits.
Certain tests in rtti-options.cpp are not really testing anything because they
are testing for the absence of -frtti option to the cc1 process. Since the cc1
process does not take -frtti option, these tests are passi
martong created this revision.
martong added reviewers: xazax.hun, a.sidorin.
Herald added subscribers: cfe-commits, dkrupp, rnkovacs.
`DeclContext` is essentially a list of `Decl`s and a lookup table (`LookupPtr`)
but these are encapsulated.
E.g. `LookupPtr` is private. `DeclContext::removeDecl`
don't understand this at all. Why would we want to do that,
> > > exactly?
> > > Just so that we can avoid creating directory structure? Why do we want
> > > to avoid that?
> >
> >
> > If clang-tidy is invoked manually, a simpler naming scheme
Author: a.sidorin
Date: Mon May 14 09:12:31 2018
New Revision: 332256
URL: http://llvm.org/viewvc/llvm-project?rev=332256&view=rev
Log:
[ASTImporter] Turn StringRefs back to std::strings to avoid use-after-free
This is a workaround for the issue in buildASTFromCodeWithArgs()
where a local buffer
jkorous created this revision.
jkorous added reviewers: arphaman, dexonsmith, rsmith, doug.gregor.
jkorous added a project: clang.
Herald added a subscriber: cfe-commits.
C++17 adds form of static_assert without string literal.
static_assert ( bool_constexpr )
Currently clang produces diagnost
xazax.hun added a comment.
The analyzer can only reason about const variables this way, right? Maybe we
should only import the initializers for such variables to have better
performance? What do you think?
Also, I wonder what happens with user-defined classes. Will the analyzer
evaluate the co
ilya-biryukov accepted this revision.
ilya-biryukov added a comment.
This revision is now accepted and ready to land.
LGTM
Comment at: lib/Format/Format.cpp:1702
// cases where the first #include is unlikely to be the main header.
- IncludeCategoryManager Categories(Style,
alexfh added a comment.
In https://reviews.llvm.org/D46159#1098018, @alexfh wrote:
> In https://reviews.llvm.org/D46159#1098000, @pfultz2 wrote:
>
> > Is someone able to merge in my changes?
>
>
> Will do.
Please rebase the patch, it doesn't apply cleanly.
https://reviews.llvm.org/D46159
_
alexfh added a comment.
In https://reviews.llvm.org/D46159#1098000, @pfultz2 wrote:
> Is someone able to merge in my changes?
Will do.
https://reviews.llvm.org/D46159
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/c
a.sidorin accepted this revision.
a.sidorin added a comment.
This revision is now accepted and ready to land.
In https://reviews.llvm.org/D46353#1086456, @martong wrote:
> > should we add this new declaration to the redeclaration chain like we do it
> > for RecordDecls?
>
> I think, on a long te
ilya-biryukov added a comment.
This LG, presuming there are no semantic changes here, just moving the code
around.
Also see the nits about `IncludeStyle` that seems to be in the wrong change...
Comment at: include/clang/Tooling/Core/HeaderIncludes.h:112
+/// priorities for hea
ilya-biryukov accepted this revision.
ilya-biryukov added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rC Clang
https://reviews.llvm.org/D46758
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.l
erik.pilkington accepted this revision.
erik.pilkington added a comment.
This revision is now accepted and ready to land.
Thanks for the explanation, LGTM.
https://reviews.llvm.org/D46747
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http:
pfultz2 added a comment.
Is someone able to merge in my changes?
https://reviews.llvm.org/D46159
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
yaxunl accepted this revision.
yaxunl added a comment.
This revision is now accepted and ready to land.
LGTM. Thanks!
Repository:
rC Clang
https://reviews.llvm.org/D43281
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.o
martong accepted this revision.
martong added a comment.
This revision is now accepted and ready to land.
LGTM!
Repository:
rC Clang
https://reviews.llvm.org/D46398
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi
HLJ2009 added inline comments.
Comment at: include/clang/Basic/Attr.td:566-567
-def Alias : Attr {
+// We do not support alias attribute on Apple platform, so we exclude the
platform.
+def Alias : Attr, TargetSpecificAttr {
let Spellings = [GCC<"alias">];
r
labath added a comment.
I've reverted the patch as it has caused miscompiles on non-trivial inputs.
I'll update the patch when I gain a better understanding of what is going on.
Repository:
rC Clang
https://reviews.llvm.org/D46685
___
cfe-commit
gt;
>
> If clang-tidy is invoked manually, a simpler naming scheme with less
> configuration options would be easier to use, in particular:
>
> 1. no the need to specify the `-store-check-profile-elide-prefix=` option;
> 2. it's easier to see all results (no need to use `find`,
r.stahl updated this revision to Diff 146609.
r.stahl marked an inline comment as done.
r.stahl added a comment.
- added tests
- updated doc and var naming
- addressed review comment
https://reviews.llvm.org/D46421
Files:
include/clang/CrossTU/CrossTranslationUnit.h
lib/CrossTU/CrossTransla
1 - 100 of 157 matches
Mail list logo
