thakis added inline comments.
Comment at: include/clang/Driver/CLCompatOptions.td:121
+def : CLFlag<"Ob0">, Alias<_SLASH_O>, AliasArgs<["b0"]>, HelpText<"Disable
function inlining">;
+def : CLFlag<"Ob1">, Alias<_SLASH_O>, AliasArgs<["b1"]>, HelpText<"Only inline
functions
vsapsai added a comment.
Ping.
https://reviews.llvm.org/D50539
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aaron.ballman added inline comments.
Comment at: clang-tidy/modernize/ConcatNestedNamespacesCheck.cpp:52
+const NamespaceContextVec ) {
+ std::ostringstream Result;
+ bool First = true;
Can this be rewritten with `llvm::for_each()` and a `Twine` so that we
aganea added a comment.
In https://reviews.llvm.org/D52193#1241056, @zturner wrote:
> The process stuff looks cool and useful independently of `/MP`. Would it be
> possible to break that into a separate patch, and add a unit test for the
> behavior of the `WaitAll` flag?
Of course, will do.
dblaikie added a comment.
Do you/what's your particular use case for this scenario? I guess this looks a
bit like Apple's format (where debug info stays in the object file and isn't
linked into the final binary), but don't expect they'd be moving to this any
time soon.
manojgupta added a comment.
lgtm. But someone more familiar with these code paths should approve.
Repository:
rC Clang
https://reviews.llvm.org/D52248
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
lebedev.ri added inline comments.
Comment at: test/Sema/gnu89.c:1-2
-// RUN: %clang_cc1 %s -std=gnu89 -pedantic -fsyntax-only -verify
+// RUN: %clang_cc1 %s -std=gnu89 -pedantic -fsyntax-only 2>&1 | FileCheck
-check-prefix=CHECK-PEDANTIC %s
+// RUN: %clang_cc1 %s -std=gnu89
srhines added inline comments.
Comment at: lib/Sema/SemaType.cpp:1682
// or via one or more typedefs."
-if (!S.getLangOpts().C99 && !S.getLangOpts().CPlusPlus
-&& TypeQuals & Result.getCVRQualifiers()) {
This is broken for C11 and C17 (even
lebedev.ri accepted this revision.
lebedev.ri added a comment.
This revision is now accepted and ready to land.
If it **no longer** crashes, i guess you can test for that?
Other than that, SG.
https://reviews.llvm.org/D52313
___
cfe-commits
aganea marked an inline comment as done.
aganea added a comment.
It seems Reid's change has done good: `cmake` is not as significant as before
in the build process. See below the improved timings:
The tests consist in a full cleanup (delete build folder), cmake to regenerate,
then a full
nickdesaulniers updated this revision to Diff 166355.
nickdesaulniers added a comment.
- move test to new file, use check-prefix for both cases
Repository:
rC Clang
https://reviews.llvm.org/D52248
Files:
lib/Sema/SemaType.cpp
test/Sema/gnu89-const.c
Index: test/Sema/gnu89-const.c
rjmccall added a comment.
`LinkageComputer` isn't actually persisted anywhere, right? And there's maybe
one computer active at once? So this compression is theoretically saving one
pointer of stack space but forcing a bunch of bit-manipulation every time these
fields are accessed.
This revision was automatically updated to reflect the committed changes.
Closed by commit rC342672: [Sema] Retain __restrict qualifiers when
substituting a reference type. (authored by epilk, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D52271?vs=166224=166334#toc
Author: epilk
Date: Thu Sep 20 11:12:24 2018
New Revision: 342672
URL: http://llvm.org/viewvc/llvm-project?rev=342672=rev
Log:
[Sema] Retain __restrict qualifiers when substituting a reference type.
Fixes rdar://43760099
Differential revision: https://reviews.llvm.org/D52271
Added:
yaxunl created this revision.
yaxunl added reviewers: kzhuravl, b-sumner, arsenm.
Herald added subscribers: t-tye, tpr, dstuttard, nhaehnle, wdng, jvesely.
https://reviews.llvm.org/D52320
Files:
include/clang/Basic/BuiltinsAMDGPU.def
lib/CodeGen/CGBuiltin.cpp
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM!
Repository:
rC Clang
https://reviews.llvm.org/D50214
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
rjmccall accepted this revision.
rjmccall added inline comments.
This revision is now accepted and ready to land.
Comment at: include/clang/AST/DeclarationName.h:46
-/// DeclarationName - The name of a declaration. In the common case,
-/// this just stores an IdentifierInfo
bruno accepted this revision.
bruno added a comment.
This revision is now accepted and ready to land.
LGTM with one more small change.
Comment at: clang/test/Modules/double-quotes.m:35
+// rdar://43692300
+#import "NotAFramework/Headers/Headers/Thing1.h"
Author: epilk
Date: Thu Sep 20 12:00:03 2018
New Revision: 342679
URL: http://llvm.org/viewvc/llvm-project?rev=342679=rev
Log:
Fix an assert in -Wquoted-include-in-framework-header
Fixes rdar://43692300
Differential revision: https://reviews.llvm.org/D52253
Added:
nickdesaulniers updated this revision to Diff 166357.
nickdesaulniers added a comment.
- add line numbers to match specific warning lines
Repository:
rC Clang
https://reviews.llvm.org/D52248
Files:
lib/Sema/SemaType.cpp
test/Sema/gnu89-const.c
Index: test/Sema/gnu89-const.c
nickdesaulniers updated this revision to Diff 166352.
nickdesaulniers added a comment.
- git-clang-format HEAD~
Repository:
rC Clang
https://reviews.llvm.org/D52248
Files:
lib/Sema/SemaType.cpp
test/Sema/gnu89.c
Index: test/Sema/gnu89.c
This revision was automatically updated to reflect the committed changes.
Closed by commit rL342679: Fix an assert in
-Wquoted-include-in-framework-header (authored by epilk, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
bruno added a comment.
Hi Volodymyr,
Thanks for working on this, really nice work with the tests! Comments below.
> - No support for 'fallthrough' in crash reproducer.
That'd be nice to have at some point to make sure we never escape into the real
fs.
> - Current way of working with modules
zturner added a comment.
The process stuff looks cool and useful independently of `/MP`. Would it be
possible to break that into a separate patch, and add a unit test for the
behavior of the `WaitAll` flag?
Repository:
rC Clang
https://reviews.llvm.org/D52193
NoQ added a comment.
You can retrieve any node from the `ASTMatcher` by `.bind()`ing sub-matchers to
string keys and later retrieving them from the `MatchResult` dictionary by
those keys. It's like a regex capturing groups (and, well, `ASTMatchers` also
support backreferences to such groups,
cmtice created this revision.
cmtice added reviewers: tejohnson, pcc.
Herald added subscribers: cfe-commits, dexonsmith, mehdi_amini.
Currently the code-model does not get saved in the module IR, so if a code
model is specified when compiling with LTO, it gets lost and is not propagated
Author: owenpan
Date: Thu Sep 20 20:46:36 2018
New Revision: 342708
URL: http://llvm.org/viewvc/llvm-project?rev=342708=rev
Log:
[clang-format] Do not merge short case labels if followed by a block.
Do not allow short case labels on a single line if the label is followed by a
left brace.
Fixes
nickdesaulniers updated this revision to Diff 166351.
nickdesaulniers added a comment.
- warn only if -pedantic, add gcc bug test cases
Repository:
rC Clang
https://reviews.llvm.org/D52248
Files:
lib/Sema/SemaType.cpp
test/Sema/gnu89.c
Index: test/Sema/gnu89.c
aganea added a comment.
> clang-cl isn't supposed to do (explicit) registry accesses when you hold it
> right (pass in -fms-compatibility-version etc). Have you seen registry access
> costs, or is that speculation?
Please see this log: F7268226: clang-cl-log.zip
owenpan abandoned this revision.
owenpan added a comment.
Closed by r342708.
Repository:
rC Clang
https://reviews.llvm.org/D52021
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Hi Eric,
The test that you added in this commit is failing on the PS4 Windows bot. Can
you please take a look?
http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast/builds/20052
FAIL: Clang :: Preprocessor/include-leading-nonalpha-suggest.c (10765 of 43992)
FWIW we're trying to reproduce here real fast and then will revert or fix
real fast.
Thanks!
-eric
On Thu, Sep 20, 2018 at 2:46 PM Eric Christopher wrote:
> Adding Jorge...
>
> On Thu, Sep 20, 2018 at 2:36 PM wrote:
>
>> Hi Eric,
>>
>> The test that you added in this commit is failing on the
Zach and I were able to find the cause.
Clang on Windows manages to find "file.h" when you #include "/file.h" and
that makes the expected diagnostic not appear. MSVC inteprets an #include
with a leading slash as an absolute path so I think we have accidentally
hit a different bug in Clang :)
One
Filed https://bugs.llvm.org/show_bug.cgi?id=39029 for the incorrect
behavior when including a path with a leading slash on Windows.
On Thu, Sep 20, 2018 at 3:35 PM Eric Christopher wrote:
> Thank you!
>
> On Thu, Sep 20, 2018, 3:34 PM Zachary Turner wrote:
>
>> Test removed in r342693.
>>
>>
sammccall added a comment.
This seems very clever, but extremely complicated - you've implemented much of
C++'s conversion logic, it's not clear to me which parts are actually necessary
to completion quality.
(Honestly this applies to expected types overall - it seems intuitively likely
that
vsk added inline comments.
Comment at: include/clang/Driver/CC1Options.td:236
+def coverage_exclude_EQ : Joined<["-"], "coverage-exclude=">,
+ Alias;
def coverage_exit_block_before_body : Flag<["-"],
"coverage-exit-block-before-body">,
marco-c wrote:
>
nickdesaulniers added inline comments.
Comment at: test/Sema/gnu89.c:1-2
-// RUN: %clang_cc1 %s -std=gnu89 -pedantic -fsyntax-only -verify
+// RUN: %clang_cc1 %s -std=gnu89 -pedantic -fsyntax-only 2>&1 | FileCheck
-check-prefix=CHECK-PEDANTIC %s
+// RUN: %clang_cc1 %s
nickdesaulniers added inline comments.
Comment at: test/Sema/gnu89.c:1-2
-// RUN: %clang_cc1 %s -std=gnu89 -pedantic -fsyntax-only -verify
+// RUN: %clang_cc1 %s -std=gnu89 -pedantic -fsyntax-only 2>&1 | FileCheck
-check-prefix=CHECK-PEDANTIC %s
+// RUN: %clang_cc1 %s
Author: zturner
Date: Thu Sep 20 15:32:51 2018
New Revision: 342693
URL: http://llvm.org/viewvc/llvm-project?rev=342693=rev
Log:
Remove failing test.
Removing on behalf of Jorge Moya. This test is broken on
Windows due to it actually being able to resolve the path. There
is an actual
Thank you!
On Thu, Sep 20, 2018, 3:34 PM Zachary Turner wrote:
> Test removed in r342693.
>
> On Thu, Sep 20, 2018 at 3:30 PM Jorge Gorbe Moya
> wrote:
>
>> Zach and I were able to find the cause.
>>
>> Clang on Windows manages to find "file.h" when you #include "/file.h" and
>> that makes the
Test removed in r342693.
On Thu, Sep 20, 2018 at 3:30 PM Jorge Gorbe Moya wrote:
> Zach and I were able to find the cause.
>
> Clang on Windows manages to find "file.h" when you #include "/file.h" and
> that makes the expected diagnostic not appear. MSVC inteprets an #include
> with a leading
MaskRay created this revision.
MaskRay added reviewers: akyrtzi, arphaman.
Herald added a subscriber: cfe-commits.
template struct B {};
template struct D : B {}; // `B` was not reported as a
reference
This patch fixes this.
Repository:
rC Clang
https://reviews.llvm.org/D52331
aaron.ballman added inline comments.
Comment at: clang-tidy/modernize/ReplaceGenericFunctorCallCheck.cpp:27-32
+ // template
+ // void f(T func) {
+ // func();
+ // ^~~
+ // ::std::invoke(func, 1)
+ Finder->addMatcher(callExpr(has(declRefExpr())).bind("functor"),
aaronpuchert added a comment.
Thanks for pointing out my error! Ignoring the implementation for a moment, do
you think this is a good idea or will it produce to many false positives?
Releasable/relockable scoped capabilities that I have seen keep track of the
status, so it makes sense, but
vsk added inline comments.
Comment at: test/CodeGen/code-coverage-filter.c:4
+// RUN: %clang_cc1 -emit-llvm -femit-coverage-data -coverage-exclude=.*\\.h %s
-o - \
+// RUN:| FileCheck -check-prefix=NOH %s
+// RUN: %clang_cc1 -emit-llvm -femit-coverage-data
steveire created this revision.
steveire added reviewers: JonasToth, george.karpenkov, shuaiwang, aaron.ballman.
Herald added subscribers: cfe-commits, Szelethus, a.sidorin, mgorny.
Conditionally compile the parts of clang-tidy which depend on the static
analyzer.
Repository:
rCTE Clang Tools
erik.pilkington created this revision.
erik.pilkington added reviewers: rsmith, rjmccall.
Herald added a subscriber: dexonsmith.
A significant number of internal C users have been complaining about the lack
of support for fixed enums. We already support this in Objective-C mode, as
well as in C
Adding Jorge...
On Thu, Sep 20, 2018 at 2:36 PM wrote:
> Hi Eric,
>
> The test that you added in this commit is failing on the PS4 Windows bot.
> Can you please take a look?
>
>
> http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast/builds/20052
>
> FAIL: Clang ::
grimar created this revision.
grimar added reviewers: dblaikie, echristo, probinson, compnerd.
Herald added subscribers: JDevlieghere, aprantl.
The DWARF5 specification says(Appendix F.1):
"The sections that do not require relocation, however, **can be written to the
relocatable object (.o)
atanasyan requested changes to this revision.
atanasyan added a comment.
This revision now requires changes to proceed.
This patch fails the following test cases:
- tools/clang/test/CodeGen/target-data.c
- tools/clang/test/Driver/mips-cs.cpp
Comment at:
lorincbalog added a comment.
Yes, `-w` suppresses all warnings (without raising an error) regardless of the
options' sequence, even if `-Wall -Werror` are present,.
Repository:
rC Clang
https://reviews.llvm.org/D51926
___
cfe-commits mailing
Author: hans
Date: Thu Sep 20 02:29:35 2018
New Revision: 342636
URL: http://llvm.org/viewvc/llvm-project?rev=342636=rev
Log:
FileCheckify test/Driver/Xarch.c
Modified:
cfe/trunk/test/Driver/Xarch.c
Modified: cfe/trunk/test/Driver/Xarch.c
URL:
sidorovd created this revision.
sidorovd added reviewers: Anastasia, yaxunl.
Herald added a subscriber: cfe-commits.
Allowed extension name (that ought to be disabled) printing in the note message.
This diagnostic was proposed here: https://reviews.llvm.org/D51341
Repository:
rC Clang
olista01 accepted this revision.
olista01 added a comment.
This revision is now accepted and ready to land.
LGTM, thanks!
https://reviews.llvm.org/D51432
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
kbobyrev added inline comments.
Comment at: clang-tools-extra/clangd/benchmarks/IndexBenchmark.cpp:69
+// This is not a *real* benchmark: it shows size of built MemIndex (in bytes).
+// Same for the next "benchmark".
ioeric wrote:
> The hack might not be
lebedev.ri added inline comments.
Comment at: clang-tidy/modernize/ReplaceGenericFunctorCallCheck.cpp:27-32
+ // template
+ // void f(T func) {
+ // func();
+ // ^~~
+ // ::std::invoke(func, 1)
+ Finder->addMatcher(callExpr(has(declRefExpr())).bind("functor"),
yvvan added a comment.
I tried that first but did not I find a way just to copy an expression (we
basically need the same expr for such case). Do you know how to properly
generate a copy of expression or some other way to get the same expression?
https://reviews.llvm.org/D52261
Author: qshanz
Date: Wed Sep 19 22:04:57 2018
New Revision: 342614
URL: http://llvm.org/viewvc/llvm-project?rev=342614=rev
Log:
[PowerPC] [Clang] Add vector int128 pack/unpack builtins
unsigned long long builtin_unpack_vector_int128 (vector int128_t, int);
vector int128_t
Anastasia added inline comments.
Comment at: include/clang/Basic/OpenCLExtensionTypes.def:27
+
+INTEL_SGAVC_TYPE(mce_payload_t, McePayload)
+INTEL_SGAVC_TYPE(ime_payload_t, ImePayload)
AlexeySotkin wrote:
> Anastasia wrote:
> > AlexeySachkov wrote:
> > >
riccibruno updated this revision to Diff 166264.
riccibruno marked 3 inline comments as done.
riccibruno added a comment.
Removed the superfluous static_assert.
Repository:
rC Clang
https://reviews.llvm.org/D52268
Files:
lib/AST/Linkage.h
Index: lib/AST/Linkage.h
Anastasia added inline comments.
Comment at: lib/Headers/opencl-c.h:26
+#if __OPENCL_C_VERSION__ >= CL_VERSION_1_2
+#ifndef cl_intel_planar_yuv
+#define cl_intel_planar_yuv
Anastasia wrote:
> @yaxunl, do you think we need to add some kind of architecture guard
kbobyrev updated this revision to Diff 166266.
kbobyrev retitled this revision from "[clangd] Add a "benchmark" for tracking
memory" to "[clangd] Add building benchmark and memory consumption tracking".
kbobyrev edited the summary of this revision.
kbobyrev added a comment.
Add symbol index
kbobyrev updated this revision to Diff 166269.
kbobyrev added a comment.
Add benchmark for `SymbolSlab` loading from file. This might be useful for
RIFF/YAML symbol loader optimizations.
https://reviews.llvm.org/D52047
Files:
clang-tools-extra/clangd/benchmarks/IndexBenchmark.cpp
sidorovd added inline comments.
Comment at: include/clang/Sema/Sema.h:8576
+
+ /// Find and extension in an extension map and return its name
+ template
Anastasia wrote:
> and extension -> an extension ?
Thanks!
Comment at:
sidorovd updated this revision to Diff 166270.
sidorovd marked an inline comment as done.
https://reviews.llvm.org/D52292
Files:
include/clang/Basic/DiagnosticSemaKinds.td
include/clang/Sema/Sema.h
lib/Sema/Sema.cpp
lib/Sema/SemaOverload.cpp
test/SemaOpenCL/extension-begin.cl
Index:
kbobyrev updated this revision to Diff 166271.
kbobyrev added a comment.
Remove `BuildMem` benchmark, which collects data about `MemIndex` building time
(which is essentially nothing and therefore not really interesting).
https://reviews.llvm.org/D52047
Files:
kbobyrev updated this revision to Diff 166278.
kbobyrev added a comment.
- Update unit tests with iterator tree string representation to comply with the
new format
- Don't mark constructor `explicit` (previously it only had one parameter)
- Fix `Limits` explanation comment (`ID > Limits[I]` ->
ebevhan added a comment.
Another ping. Anyone up for reviewing this patch?
Repository:
rC Clang
https://reviews.llvm.org/D51211
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Anastasia added inline comments.
Comment at: include/clang/Sema/Sema.h:8576
+
+ /// Find and extension in an extension map and return its name
+ template
and extension -> an extension ?
Comment at: lib/Sema/Sema.cpp:1856
+std::string
kbobyrev created this revision.
kbobyrev added reviewers: ioeric, sammccall, ilya-biryukov.
kbobyrev added a project: clang-tools-extra.
Herald added subscribers: kadircet, arphaman, jkorous, MaskRay, mgorny.
kbobyrev edited the summary of this revision.
This patch implements Variable-length Byte
christylee accepted this revision.
christylee added a comment.
This revision is now accepted and ready to land.
Thanks for the fix!
Repository:
rC Clang
https://reviews.llvm.org/D52280
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
pfultz2 added a comment.
This needs a test when calling in a `constexpr` function. I believe
`std::invoke` is not `constepxr`, so a function object call in a `constexpr`
function should not suggest `std::invoke`.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D52281
kadircet created this revision.
kadircet added a reviewer: ilya-biryukov.
Herald added a subscriber: cfe-commits.
Repository:
rC Clang
https://reviews.llvm.org/D52301
Files:
include/clang/AST/PrettyPrinter.h
include/clang/Basic/DiagnosticSemaKinds.td
include/clang/Sema/Sema.h
jkorous created this revision.
jkorous added reviewers: arphaman, vsapsai, sammccall, ilya-biryukov.
jkorous added a project: clang.
Herald added subscribers: cfe-commits, dexonsmith, eraman.
Destructors don't have return type "void", they don't have any return type at
all.
Repository:
rC
thakis added a comment.
In https://reviews.llvm.org/D52266#1240304, @hans wrote:
> Sorry, I didn't realize we both set off to do this in parallel. I've
> incorporated your changes into my patch.
No worries, I didn't do anything I wouldn't have done for reviewing this :-)
Thoughts on "As far
sammccall added a comment.
When you're *calling* a destructor, I believe the expression does have type
void. Are we sure this is incorrect?
Calling a destructor is really unusual though. IIRC we decided to just not show
them in clangd in member context (maybe this is broken or was never
aaron.ballman accepted this revision.
aaron.ballman added a comment.
LGTM!
Repository:
rC Clang
https://reviews.llvm.org/D52280
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jkorous added a comment.
You might be right - I am assuming here that we want consistent behaviour
between constructors and destructors.
IIUC ctors are currently skipped in code completions (in most cases) but they
also don't have any type associated while result of their call definitely has
kbobyrev added a comment.
Sorry, I didn't get time to review the patch properly, these are few stylistic
comments. Hopefully, I'll be able to give more feedback when I get more time.
Comment at: clang-tidy/readability/IsolateDeclCheck.cpp:21
+
+#define PRINT_DEBUG 1
+
yaxunl added a comment.
ping
https://reviews.llvm.org/D51809
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jkorous added a comment.
Sorry my bad. You are right, we aren't showing destructors in clangd for normal
classes. The case where I noticed is kind of a corner case with template class.
ilya-biryukov created this revision.
ilya-biryukov added reviewers: ioeric, sammccall, simark.
Herald added subscribers: kadircet, arphaman, jkorous, MaskRay.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D52311
Files:
clangd/ClangdLSPServer.cpp
clangd/ClangdServer.cpp
sammccall added a comment.
In https://reviews.llvm.org/D52308#1240642, @jkorous wrote:
> You might be right - I am assuming here that we want consistent behaviour
> between constructors and destructors.
>
> IIUC ctors are currently skipped in code completions (in most cases) but they
> also
ilya-biryukov planned changes to this revision.
ilya-biryukov added a comment.
Posted to make sure it's visible that I've started doing this.
Still need to update the tests and check for the capability from the client
(and fallback to SymbolInformation if client does not support the new
hans added a comment.
Sorry, I didn't realize we both set off to do this in parallel. I've
incorporated your changes into my patch.
Comment at: test/Driver/Xarch.c:5
+// RUN: not grep ' "-O3" ' %t.log
+// RUN: grep "argument unused during compilation: '-Xarch_i386 -O3'"
hans updated this revision to Diff 166252.
hans added a comment.
Uploading new diff.
https://reviews.llvm.org/D52266
Files:
include/clang/Driver/CLCompatOptions.td
lib/Driver/ToolChains/MSVC.cpp
test/Driver/Xarch.c
Index: test/Driver/Xarch.c
Author: svenvh
Date: Thu Sep 20 03:07:27 2018
New Revision: 342638
URL: http://llvm.org/viewvc/llvm-project?rev=342638=rev
Log:
[OpenCL] Diagnose redundant address space conversion
Add a warning if a parameter with a named address space is passed
to a to_addr builtin.
For example:
int i;
This revision was automatically updated to reflect the committed changes.
Closed by commit rL342638: [OpenCL] Diagnose redundant address space conversion
(authored by svenvh, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
kbobyrev updated this revision to Diff 166240.
kbobyrev marked 4 inline comments as done.
https://reviews.llvm.org/D52047
Files:
clang-tools-extra/clangd/benchmarks/IndexBenchmark.cpp
clang-tools-extra/clangd/index/SymbolYAML.cpp
clang-tools-extra/clangd/index/dex/Dex.cpp
Index:
marco-c added inline comments.
Comment at: include/clang/Driver/CC1Options.td:236
+def coverage_exclude_EQ : Joined<["-"], "coverage-exclude=">,
+ Alias;
def coverage_exit_block_before_body : Flag<["-"],
"coverage-exit-block-before-body">,
calixte wrote:
>
atanasyan created this revision.
atanasyan added reviewers: rsmith, rnk, zturner.
Herald added subscribers: jrtc27, arichardson, sdardis.
All MIPS target architectures can be divided into four categories: 32-bit
little-endian, 32-bit big-endian, 64-bit little-endian, and 64-bit big-endian.
If,
kbobyrev updated this revision to Diff 166241.
kbobyrev marked an inline comment as done.
kbobyrev retitled this revision from "[dexp] Allow users to dump JSON
representations of fuzzy find requests" to "[dexp] Dump JSON representations of
fuzzy find requests".
kbobyrev edited the summary of
sammccall added a comment.
In https://reviews.llvm.org/D52308#1240680, @jkorous wrote:
> Sorry my bad. You are right, we aren't showing destructors in clangd for
> normal classes. The case where I noticed is kind of a corner case with
> template class.
>
>
>
shuaiwang added inline comments.
Comment at: unittests/Analysis/ExprMutationAnalyzerTest.cpp:67
+if (const auto *DRE = dyn_cast(E)) {
+ if (DRE->getNameInfo().getAsString()[0] == 'p')
+Finder = PointeeMutationFinder;
JonasToth wrote:
> shuaiwang
JonasToth added inline comments.
Comment at: unittests/Analysis/ExprMutationAnalyzerTest.cpp:67
+if (const auto *DRE = dyn_cast(E)) {
+ if (DRE->getNameInfo().getAsString()[0] == 'p')
+Finder = PointeeMutationFinder;
shuaiwang wrote:
> JonasToth
juliehockett created this revision.
juliehockett added reviewers: jakehehrlich, leonardchan, lebedev.ri.
juliehockett added a project: clang-tools-extra.
Don't try to parse base classes for declarations that are not definitions
(segfaults, as there is no DefinitionData to access).
simark added a comment.
Ohh awesome, I didn't know the LSP supported that. I'll try it it Theia when I
have time.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D52311
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
JonasToth added inline comments.
Comment at: clang-tidy/utils/TypeTraits.cpp:47
+
+ // Do not consider "expensive to copy" types not greater than a pointer
+ if (Context.getTypeSize(Type) <= Context.getTypeSize(Context.VoidPtrTy))
Please make that comment a
Author: echristo
Date: Thu Sep 20 10:22:43 2018
New Revision: 342668
URL: http://llvm.org/viewvc/llvm-project?rev=342668=rev
Log:
Add testcases for r342667.
Added:
cfe/trunk/test/Preprocessor/include-leading-nonalpha-no-suggest.c
This revision was automatically updated to reflect the committed changes.
Closed by commit rL342667: r342177 introduced a hint in cases where an
#included file is not found. It… (authored by echristo, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
1 - 100 of 116 matches
Mail list logo