Thanks, looks good.
On Mon, 30 Apr 2018, 23:52 Chandler Carruth via cfe-commits, <
cfe-commits@lists.llvm.org> wrote:
> I adjusted the test case in r331245 - it was failing for me and most of
> the build bots. Please check that I didn't miss anything.
>
> On Mon, Apr 30, 2018 at 10:06 PM Richard
theraven added inline comments.
Comment at: lib/CodeGen/CGObjCGNU.cpp:439
+ ArrayRef IvarOffsets,
+ ArrayRef IvarAlign,
+ ArrayRef IvarOwnership);
DHowett-MSFT wrote:
> While we'r
chill added a comment.
Ping?
Repository:
rC Clang
https://reviews.llvm.org/D46013
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
lebedev.ri added a comment.
Global question: you are using `NamedDecl::getNameAsString()`, and passing it
as `StringRef`.
You have looked at this with ASAN, and it's ok?
Also, have you considered using the `NamedDecl::getName()`, which already
returns `StringRef`.?
Comment a
This revision was automatically updated to reflect the committed changes.
Closed by commit rL331249: [X86] directstore and movdir64b intrinsics (authored
by GBuella, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D45984?vs=143718&id=14
Author: gbuella
Date: Tue May 1 03:05:42 2018
New Revision: 331249
URL: http://llvm.org/viewvc/llvm-project?rev=331249&view=rev
Log:
[X86] directstore and movdir64b intrinsics
Reviewers: spatel, craig.topper, RKSimon
Reviewed By: craig.topper
Differential Revision: https://reviews.llvm.org/D45
lebedev.ri added inline comments.
Comment at: clang-doc/BitcodeWriter.cpp:382
+ emitRecord(R.USR, REFERENCE_USR);
+ emitRecord(R.Name, REFERENCE_NAME);
+ emitRecord((unsigned)R.RefType, REFERENCE_TYPE);
>>! In D46281#1083806, @lebedev.ri wrote:
> Global questi
Hi Nico,
This broke armv7 and aarch64 bots:
-
http://lab.llvm.org:8011/builders/libcxx-libcxxabi-libunwind-armv7-linux/builds/87
-
http://lab.llvm.org:8011/builders/libcxx-libcxxabi-libunwind-aarch64-linux/builds/1304
Would you please investigate?
--
Maxim Kuvyrkov
www.linaro.org
> On Apr 3
rsandifo-arm created this revision.
Herald added subscribers: cfe-commits, rengolin.
[ Not intended for commit. I'm just posting it to back up an
RFC on cfe-dev. ]
This patch is a hack to add two new "sizeless builtin" types __SVInt8_t
and __SVInt16_t. The real SVE implementation looks nothi
rsandifo-arm created this revision.
Herald added subscribers: cfe-commits, kristof.beyls, rengolin.
Herald added a reviewer: javed.absar.
`[ Not intended for commit. I'm just posting it to back up an
RFC on cfe-dev. ]
This patch adds the concept of "sizeless" types to core C and C++.
It's a p
tzik, can you take a look what's up on those bots?
On Tue, May 1, 2018, 5:48 AM Maxim Kuvyrkov
wrote:
> Hi Nico,
>
> This broke armv7 and aarch64 bots:
> -
> http://lab.llvm.org:8011/builders/libcxx-libcxxabi-libunwind-armv7-linux/builds/87
> -
> http://lab.llvm.org:8011/builders/libcxx-libcxxab
mikhail.ramalho marked an inline comment as done.
mikhail.ramalho added a comment.
Ping.
https://reviews.llvm.org/D36610
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mikhail.ramalho added a comment.
Hi,
> After thinking about this a bit, and use cases like rename and
> find-declaration that could be USR based, I think including some location
> information is the right way to go, which I think is the current behavior.
What do you man by location information
Author: abataev
Date: Tue May 1 07:09:46 2018
New Revision: 331261
URL: http://llvm.org/viewvc/llvm-project?rev=331261&view=rev
Log:
[OPENMP] Emit template instatiation|specialization functions for
devices.
If the function is an instantiation|specialization of the template and
is used in the dev
Author: erichkeane
Date: Tue May 1 07:16:15 2018
New Revision: 331263
URL: http://llvm.org/viewvc/llvm-project?rev=331263&view=rev
Log:
Add Microsoft Mangling for OpenCL Half Type
Half-type mangling is accomplished following the method introduced by Erich
Keane for mangling _Float16. Updated th
This revision was automatically updated to reflect the committed changes.
Closed by commit rL331263: Add Microsoft Mangling for OpenCL Half Type
(authored by erichkeane, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D46131?vs=144139&i
This revision was automatically updated to reflect the committed changes.
Closed by commit rC331263: Add Microsoft Mangling for OpenCL Half Type
(authored by erichkeane, committed by ).
Repository:
rC Clang
https://reviews.llvm.org/D46131
Files:
lib/AST/MicrosoftMangle.cpp
test/CodeGenOpe
benhamilton marked 2 inline comments as done.
benhamilton added a comment.
Thanks for the review. Added another test.
Comment at: test/clang-tidy/google-runtime-int.cpp:77
+__attribute__((__format__ (__printf__, 1, 2)))
+void myprintf(const char* s, ...);
+
ale
benhamilton updated this revision to Diff 144717.
benhamilton added a comment.
Add another test for printf without attribute
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D46293
Files:
clang-tidy/google/IntegerTypesCheck.cpp
test/clang-tidy/google-runtime-int.cpp
Index: t
zinovy.nis added a comment.
Gentle ping.
https://reviews.llvm.org/D45932
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: benhamilton
Date: Tue May 1 07:48:51 2018
New Revision: 331267
URL: http://llvm.org/viewvc/llvm-project?rev=331267&view=rev
Log:
[clang-tidy] Add Apple prefix acronyms to objc-property-declaration
Summary:
This adds a few common Apple first-party API prefixes as acronyms to
`objc-propert
Author: benhamilton
Date: Tue May 1 07:48:54 2018
New Revision: 331268
URL: http://llvm.org/viewvc/llvm-project?rev=331268&view=rev
Log:
[clang-tidy/google-runtime-int] Allow passing non-bitwidth types to
printf()-style APIs
Summary:
The `google-runtime-int` check currently fires on calls like:
This revision was automatically updated to reflect the committed changes.
benhamilton marked an inline comment as done.
Closed by commit rL331268: [clang-tidy/google-runtime-int] Allow passing
non-bitwidth types to printf()… (authored by benhamilton, committed by ).
Herald added a subscriber: llvm
This revision was automatically updated to reflect the committed changes.
Closed by commit rL331267: [clang-tidy] Add Apple prefix acronyms to
objc-property-declaration (authored by benhamilton, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews.llvm.o
khuttun created this revision.
khuttun added a reviewer: alexfh.
Herald added subscribers: xazax.hun, mgorny.
Warns on std::map and std::unordered_map lookups done with operator[].
Inserting values with operator[] is still allowed.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/
jkorous marked an inline comment as done.
jkorous added a comment.
Volodymyr, could you please confirm that the non-anonymous vs non-inline logic
makes sense to you?
Comment at: Sema/SemaDecl.cpp:4659-4661
+ (isa(OwnerScope) ||
+ (isa(OwnerScope) &&
+
jkorous added inline comments.
Comment at: Sema/SemaDecl.cpp:4651-4653
+
+ DeclContext *OwnerScope = Owner->getRedeclContext();
+
vsapsai wrote:
> I think the code style favours avoiding excessive vertical whitespace and I
> don't feel like this statement i
zinovy.nis updated this revision to Diff 144727.
zinovy.nis added a comment.
- Applied Alexander's changes.
https://reviews.llvm.org/D45927
Files:
clang-tidy/checks/modernize-use-auto.rst
clang-tidy/modernize-use-auto-min-type-name-length.cpp
modernize/UseAutoCheck.cpp
Index: clang-tidy/
khuttun added a comment.
This checker is inspired by Louis Brandy's excellent CppCon 2017 talk
(https://www.youtube.com/watch?v=lkgszkPnV8g) where he describes the
side-effects of std::map::operator[] as being one of the most common source of
bugs in Facebook's C++ code. In the talk he mentions
lebedev.ri added inline comments.
Comment at: clang-tidy/bugprone/MapSubscriptOperatorLookupCheck.cpp:25
+ hasName("operator[]"),
+ ofClass(hasAnyName("::std::map", "::std::unordered_map",
+ hasParent(implicitCastExpr(
I wou
On Mon, Apr 30, 2018 at 10:02 PM, Richard Smith via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: rsmith
> Date: Mon Apr 30 22:02:45 2018
> New Revision: 331244
>
> URL: http://llvm.org/viewvc/llvm-project?rev=331244&view=rev
> Log:
> Implement P0482R2, support for char8_t type.
>
> T
aprantl created this revision.
aprantl added a reviewer: rsmith.
Herald added subscribers: kbarton, aheejin, eraman, jgravelle-google, sbc100,
nhaehnle, nemanjai, dschuff, jfb, jholewinski, klimek.
Note that I did the same thing for LLVM this morning
(https://reviews.llvm.org/D46290).
We've bee
srhines added a comment.
Looks fine. Please wait for a more experienced person to review this, however.
Thanks for adding this feature.
Repository:
rC Clang
https://reviews.llvm.org/D46300
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
malcolm.parsons updated this revision to Diff 144742.
malcolm.parsons added a comment.
Add test
Repository:
rC Clang
https://reviews.llvm.org/D45611
Files:
lib/StaticAnalyzer/Core/HTMLDiagnostics.cpp
test/Coverage/html-multifile-diagnostics.c
Index: test/Coverage/html-multifile-diagnos
lebedev.ri created this revision.
lebedev.ri added reviewers: craig.topper, asbirlea, rnk, GGanesh, andreadb.
Herald added subscribers: Sanitizers, JDevlieghere, arichardson, aprantl,
dberris, sdardis.
The compiler-rt side of https://reviews.llvm.org/D46314
I have discovered an issue by accident
svenvh updated this revision to Diff 144745.
svenvh edited the summary of this revision.
svenvh added a comment.
Dropped the `asm` and `goto` restrictions from this patch for now until we have
clarification from Khronos.
Applied the `thread_local` restrictions to OpenCL C too.
https://reviews.
juliehockett updated this revision to Diff 144747.
juliehockett marked 3 inline comments as done.
juliehockett added a comment.
Addressing a fixme and updating tests
https://reviews.llvm.org/D46281
Files:
clang-doc/BitcodeWriter.cpp
clang-doc/BitcodeWriter.h
clang-doc/Representation.h
c
bruno closed this revision.
bruno marked an inline comment as done.
bruno added a comment.
Landed in r331232 & r331233.
Repository:
rC Clang
https://reviews.llvm.org/D46165
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm
juliehockett added inline comments.
Comment at: clang-doc/BitcodeWriter.cpp:382
+ emitRecord(R.USR, REFERENCE_USR);
+ emitRecord(R.Name, REFERENCE_NAME);
+ emitRecord((unsigned)R.RefType, REFERENCE_TYPE);
lebedev.ri wrote:
> >>! In D46281#1083806, @lebedev.ri
Eugene.Zelenko added inline comments.
Comment at: clang-tidy/bugprone/MapSubscriptOperatorLookupCheck.cpp:36
+ if (const auto *Matched = Result.Nodes.getNodeAs("match")) {
+diag(Matched->getLocStart(), "do not use operator[] for map lookup")
+<< Matched->getSourceRan
LegalizeAdulthood accepted this revision.
LegalizeAdulthood added a comment.
This revision is now accepted and ready to land.
Other than a few minor nits, ship it.
Comment at: clang-tidy/modernize/RawStringLiteralCheck.cpp:110
+
+ // Upper ASCII are disallowed too.
+ for (uns
craig.topper added inline comments.
Comment at: lib/builtins/cpu_model.c:421
*Subtype = AMDFAM15H_BDVER2;
break; // "bdver2"; 10h-1Fh: Piledriver
}
Update the comment please.
Repository:
rCRT Compiler Runtime
https://reviews.llvm.org/D4632
zinovy.nis added inline comments.
Comment at: clang-tidy/modernize/RawStringLiteralCheck.cpp:111
+ // Upper ASCII are disallowed too.
+ for (unsigned char C = 0xFFu; C >= 0x80u; --C)
+DisallowedChars.set(C);
LegalizeAdulthood wrote:
> Why does this loop go
lebedev.ri accepted this revision.
lebedev.ri added a comment.
This revision is now accepted and ready to land.
I can't really comment on the BC change since i don't think anything
here makes use of that yet, but the code changes look ok to me.
LGTM.
Comment at: clang-doc/Bitc
LegalizeAdulthood added inline comments.
Comment at: clang-tidy/modernize/RawStringLiteralCheck.cpp:111
+ // Upper ASCII are disallowed too.
+ for (unsigned char C = 0xFFu; C >= 0x80u; --C)
+DisallowedChars.set(C);
zinovy.nis wrote:
> LegalizeAdulthood wrot
lebedev.ri updated this revision to Diff 144749.
lebedev.ri marked an inline comment as done.
lebedev.ri added a comment.
Update comment, too.
Repository:
rL LLVM
https://reviews.llvm.org/D46323
Files:
lib/builtins/cpu_model.c
Index: lib/builtins/cpu_model.c
=
craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rL LLVM
https://reviews.llvm.org/D46323
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm
lebedev.ri added a comment.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85591
Repository:
rL LLVM
https://reviews.llvm.org/D46323
___
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 rL331292: Update existed CodeGen TBAA tests (authored by
Danil, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D44616?vs=138874&id=144758#toc
R
zinovy.nis created this revision.
zinovy.nis added reviewers: george.karpenkov, alexfh, hokein.
zinovy.nis added a project: clang-tools-extra.
Herald added subscribers: cfe-commits, a.sidorin, xazax.hun.
This macro is widely used in many well-known projects, ex. Chromium.
But it's not set for cla
Hi,
> After thinking about this a bit, and use cases like rename and
> find-declaration that could be USR based, I think including some location
> information is the right way to go, which I think is the current behavior.
>
>
What do you man by location information? Only the filename or filename
Author: danil
Date: Tue May 1 11:14:36 2018
New Revision: 331292
URL: http://llvm.org/viewvc/llvm-project?rev=331292&view=rev
Log:
Update existed CodeGen TBAA tests
Reviewers: hfinkel, kosarev, rjmccall
Reviewed By: rjmccall
Differential Revision: https://reviews.llvm.org/D44616
Modified:
zinovy.nis marked an inline comment as done.
zinovy.nis added inline comments.
Comment at: test/clang-tidy/modernize-raw-string-literal.cpp:44
+char const *const MultibyteSnowman("\xE2\x98\x83");
+// CHECK-FIXES: {{^}}char const *const MultibyteSnowman("\xE2\x98\x83");{{$}}
---
beanz added a reviewer: bruno.
beanz added a subscriber: bruno.
beanz added a comment.
@dexonsmith is often super busy, so @bruno may be able to weigh in instead.
Repository:
rC Clang
https://reviews.llvm.org/D45639
___
cfe-commits mailing list
c
This revision was automatically updated to reflect the committed changes.
Closed by commit rL331295: [compiler-rt][X86][AMD][Bulldozer] Fix Bulldozer
Model 2 detection. (authored by lebedevri, committed by ).
Herald added a subscriber: delcypher.
Changed prior to commit:
https://reviews.llvm.or
Author: compnerd
Date: Tue May 1 11:40:42 2018
New Revision: 331296
URL: http://llvm.org/viewvc/llvm-project?rev=331296&view=rev
Log:
Driver: fix an assertion with `-print-prog-name=`
Fix an assertion when -print-prog-name= is invoked without parameter.
Returns an empty string.
Patch by Christi
compnerd closed this revision.
compnerd added a comment.
SVN r331296
Repository:
rC Clang
https://reviews.llvm.org/D45814
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: zinovy.nis
Date: Tue May 1 11:46:32 2018
New Revision: 331297
URL: http://llvm.org/viewvc/llvm-project?rev=331297&view=rev
Log:
[clang-tidy][modernize-raw-string-literal] Don't replace upper ASCII with raw
literals
It's useless and not safe to replace UTF-8 encoded with escaped ASCII to
Author: rsmith
Date: Tue May 1 11:50:15 2018
New Revision: 331299
URL: http://llvm.org/viewvc/llvm-project?rev=331299&view=rev
Log:
Fix bogus MSVC char8_t mangling.
This appears to have been caused by a bad automatic svn merge with r330225
attaching the 'case' label to the wrong block of code. :
On 1 May 2018 at 08:55, David Majnemer via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> On Mon, Apr 30, 2018 at 10:02 PM, Richard Smith via cfe-commits <
> cfe-commits@lists.llvm.org> wrote:
>
>> Author: rsmith
>> Date: Mon Apr 30 22:02:45 2018
>> New Revision: 331244
>>
>> URL: http://llvm.
rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
Alrght, LGTM.
https://reviews.llvm.org/D46022
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/l
efriedma added a comment.
I'd like to see some tests for __attribute((packed)).
Comment at: lib/CodeGen/TargetInfo.cpp:5787
+ } else {
+ TyAlign = getContext().getTypeAlign(Ty) ;
+ }
Whitespace.
Repository:
rC Clang
https://reviews.llvm.org/D46013
george.karpenkov accepted this revision.
george.karpenkov added a comment.
LGTM
Repository:
rC Clang
https://reviews.llvm.org/D45611
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
zinovy.nis closed this revision.
zinovy.nis added a comment.
Fixed in git-svn-id:
https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@331297
91177308-0d34-0410-b5e6-96231b3b80d8
https://reviews.llvm.org/D45932
___
cfe-commits mailing list
c
zinovy.nis added a comment.
https://reviews.llvm.org/rCTE331297
https://reviews.llvm.org/D45932
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
lebedev.ri added a comment.
In https://reviews.llvm.org/D45931#1083192, @lebedev.ri wrote:
> Thank you for looking at this.
>
> In https://reviews.llvm.org/D45931#1083184, @alexfh wrote:
>
> > From a user's perspective I'd probably prefer a different behavior of
> > checks profiling with multipl
Author: smeenai
Date: Tue May 1 13:38:05 2018
New Revision: 331305
URL: http://llvm.org/viewvc/llvm-project?rev=331305&view=rev
Log:
[ARM] Remove redundant #if in test. NFC
Both sides of this #if #include the same file. Drop the #if, leaving only the
#include.
Patch by Matt Glazar.
Differenti
This revision was automatically updated to reflect the committed changes.
Closed by commit rC331305: [ARM] Remove redundant #if in test. NFC (authored by
smeenai, committed by ).
Herald added a reviewer: javed.absar.
Changed prior to commit:
https://reviews.llvm.org/D45779?vs=142969&id=144775#t
This revision was automatically updated to reflect the committed changes.
Closed by commit rL331306: [libclang] Fix the type of 'int (Foo);'
(authored by smeenai, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews.llvm.org/D45713
Files:
cfe/trunk/te
Author: smeenai
Date: Tue May 1 13:45:25 2018
New Revision: 331306
URL: http://llvm.org/viewvc/llvm-project?rev=331306&view=rev
Log:
[libclang] Fix the type of 'int (Foo);'
libclang exposes the type of 'int (Foo);' (a global variable of type int
called Foo) as CXType_Unexposed. This is because C
craig.topper created this revision.
craig.topper added a reviewer: rnk.
Every other builtin that uses TARGET_HEADER_BUILTIN has an 'h' in their
attribute string. Assume this is an oversight here.
https://reviews.llvm.org/D46332
Files:
include/clang/Basic/BuiltinsX86.def
Index: include/clan
jfb accepted this revision.
jfb added a comment.
This revision is now accepted and ready to land.
On the WebKit side this lgtm. Let's leave some time for clang-format folks to
chime in.
Repository:
rC Clang
https://reviews.llvm.org/D46024
___
cf
vsapsai updated this revision to Diff 144802.
vsapsai added a comment.
- Fix FileSkipped adding system headers when it shouldn't.
https://reviews.llvm.org/D30881
Files:
clang/lib/Frontend/DependencyFile.cpp
clang/test/Frontend/Inputs/SystemHeaderPrefix/with-header-guard.h
clang/test/Front
vsapsai marked 2 inline comments as done.
vsapsai added inline comments.
Comment at: lib/Frontend/DependencyFile.cpp:191
+ const Token &FilenameTok,
+ SrcMgr::CharacteristicKind FileType) override;
+
bruno wrote:
> Is there any
bruno accepted this revision.
bruno added a comment.
This revision is now accepted and ready to land.
LGTM, thanks!
Repository:
rC Clang
https://reviews.llvm.org/D34848
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org
lebedev.ri added inline comments.
Comment at: clang-doc/BitcodeWriter.cpp:382
+ emitRecord(R.USR, REFERENCE_USR);
+ emitRecord(R.Name, REFERENCE_NAME);
+ emitRecord((unsigned)R.RefType, REFERENCE_TYPE);
lebedev.ri wrote:
> juliehockett wrote:
> > lebedev.ri wr
vsapsai added a comment.
In https://reviews.llvm.org/D45884#1084060, @jkorous wrote:
> Volodymyr, could you please confirm that the non-anonymous vs non-inline
> logic makes sense to you?
Looks correct to me that we are checking non-anonymous namespaces. Inline
namespaces are somewhat similar
bruno accepted this revision.
bruno added a comment.
This revision is now accepted and ready to land.
Thanks for the detailed answers. LGTM
https://reviews.llvm.org/D30881
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org
spatel added a comment.
Seems right, but I don't know much about nothrow. Does some other attribute on
these intrinsics suppress the possibility of throwing an exception? Ie, is
there some way to test this?
I can't seem to produce catch/landingpad IR even after putting an intrinsic in
a 'try'
Author: dyung
Date: Tue May 1 16:32:09 2018
New Revision: 331318
URL: http://llvm.org/viewvc/llvm-project?rev=331318&view=rev
Log:
This test fails if there is no integrated assembler, so change the -c option to
-S as it is not important to the test and allows it to pass when there is no
integra
Author: vsapsai
Date: Tue May 1 16:59:33 2018
New Revision: 331319
URL: http://llvm.org/viewvc/llvm-project?rev=331319&view=rev
Log:
Track skipped files in dependency scanning.
It's possible for a header to be a symlink to another header. In this
case both will be represented by clang::FileEntry
This revision was automatically updated to reflect the committed changes.
vsapsai marked an inline comment as done.
Closed by commit rL331319: Track skipped files in dependency scanning.
(authored by vsapsai, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
http
rsmith accepted this revision.
rsmith added inline comments.
Comment at: Sema/SemaDecl.cpp:4659-4661
+ (isa(OwnerScope) ||
+ (isa(OwnerScope) &&
+cast(OwnerScope)->getDeclName( {
jkorous wrote:
> rsmith wrote:
> > vsapsai wrote:
efriedma added a comment.
You could write a test like
`static_assert(noexcept(__builtin_ia32_bzhi_di(0,0)), "");`. I don't think it
has any other effect.
Repository:
rC Clang
https://reviews.llvm.org/D46328
___
cfe-commits mailing list
cfe-com
aaron.ballman accepted this revision.
aaron.ballman added a comment.
Aside from the issue @alexfh raised, this LGTM.
Repository:
rC Clang
https://reviews.llvm.org/D46233
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.or
aaron.ballman requested changes to this revision.
aaron.ballman added a comment.
This revision now requires changes to proceed.
Missing Sema tests that the attribute only appertains to functions, accepts no
arguments, etc.
Comment at: include/clang/Basic/Attr.td:1494
+def NoSt
phosek updated this revision to Diff 144824.
phosek marked an inline comment as done.
Repository:
rC Clang
https://reviews.llvm.org/D45997
Files:
clang/cmake/caches/Fuchsia-stage2.cmake
clang/cmake/caches/Fuchsia.cmake
Index: clang/cmake/caches/Fuchsia.cmake
=
Author: bruno
Date: Tue May 1 19:25:03 2018
New Revision: 331322
URL: http://llvm.org/viewvc/llvm-project?rev=331322&view=rev
Log:
[Modules] Allow @import to reach submodules in private module maps
A @import targeting a top level module from a private module map file
(@import Foo_Private), would
Author: tstellar
Date: Tue May 1 20:23:58 2018
New Revision: 331330
URL: http://llvm.org/viewvc/llvm-project?rev=331330&view=rev
Log:
Creating release candidate final from release_502 branch
Added:
libcxxabi/tags/RELEASE_502/final/
- copied from r331329, libcxxabi/branches/release_50/
Author: tstellar
Date: Tue May 1 20:23:55 2018
New Revision: 331329
URL: http://llvm.org/viewvc/llvm-project?rev=331329&view=rev
Log:
Creating release candidate final from release_502 branch
Added:
libcxx/tags/RELEASE_502/final/ (props changed)
- copied from r331328, libcxx/branches/
Author: tstellar
Date: Tue May 1 20:24:29 2018
New Revision: 331336
URL: http://llvm.org/viewvc/llvm-project?rev=331336&view=rev
Log:
Creating release candidate final from release_502 branch
Added:
libunwind/tags/RELEASE_502/final/ (props changed)
- copied from r331335, libunwind/bra
craig.topper added a subscriber: eli.friedman.
craig.topper added a comment.
@eli.friedman is that test worth doing? Would we do it for every builtin?
Repository:
rC Clang
https://reviews.llvm.org/D46328
___
cfe-commits mailing list
cfe-commits@l
sammccall added a comment.
In https://reviews.llvm.org/D42966#1083954, @mikhail.ramalho wrote:
> Hi,
>
> > After thinking about this a bit, and use cases like rename and
> > find-declaration that could be USR based, I think including some location
> > information is the right way to go, which I
93 matches
Mail list logo