ahatanak created this revision.
ahatanak added a reviewer: rjmccall.
Herald added a subscriber: dexonsmith.
Currently IRGen doesn't handle variables captured by a block correctly when the
variable is captured by reference by a lambda that encloses the block.
For example, in the following code, t
tstellar created this revision.
tstellar added a reviewer: Sunil_Srivastava.
The check lines for this test assume the old c++11 ABI.
This resolved PR33132.
Repository:
rT test-suite
https://reviews.llvm.org/D51021
Files:
ABI-Testsuite/test/mangling/test.xpp
Index: ABI-Testsuite/test/man
tstellar created this revision.
tstellar added a reviewer: homerdin.
Repository:
rT test-suite
https://reviews.llvm.org/D51020
Files:
MultiSource/Benchmarks/DOE-ProxyApps-C++/CLAMR/MallocPlus.h
Index: MultiSource/Benchmarks/DOE-ProxyApps-C++/CLAMR/MallocPlus.h
=
rsmith added a comment.
Another possibility would be to add a command-line argument to set the version
of GCC that clang pretends to be. We have a similar mechanism for our MSVC
compatibility mode already.
Repository:
rC Clang
https://reviews.llvm.org/D51011
_
rsmith added a comment.
We are not fully compatible with any version of GCC later than 4.2, as we do
not implement `__builtin_va_arg_pack` / `__builtin_va_arg_pack_len`. These
builtins are used by the glibc headers if we claim to be GCC >= 4.3 (at least
according to
http://clang.llvm.org/docs/
metzman added inline comments.
Comment at: lib/Driver/ToolChains/MSVC.cpp:373
+ TC.getCompilerRTArgString(Args, "fuzzer",
false)));
+CmdArgs.push_back(Args.MakeArgString("-debug"));
+// Prevent the linker from padding sections we use for instr
morehouse added a comment.
LGTM
Comment at: lib/Driver/ToolChains/MSVC.cpp:373
+ TC.getCompilerRTArgString(Args, "fuzzer",
false)));
+CmdArgs.push_back(Args.MakeArgString("-debug"));
+// Prevent the linker from padding sections we use for in
This revision was automatically updated to reflect the committed changes.
Closed by commit rC340235: [WebAssembly] Revert type of wake count in
atomic.wake to i32 (authored by aheejin, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D51013?vs=161615&id=161620#toc
Repository:
Author: aheejin
Date: Mon Aug 20 16:49:34 2018
New Revision: 340235
URL: http://llvm.org/viewvc/llvm-project?rev=340235&view=rev
Log:
[WebAssembly] Revert type of wake count in atomic.wake to i32
Summary:
We decided to revert this from i64 to i32 in Nov 28 CG meeting. Fixes
PR38632.
Reviewers: d
metzman added a reviewer: morehouse.
metzman added a comment.
Matt could you please take a look at this?
I'll add rnk as reviewer once you sign off, since he is a CODE_OWNER for
Windows in Clang.
Thanks!
Repository:
rC Clang
https://reviews.llvm.org/D51008
__
aheejin created this revision.
aheejin added a reviewer: dschuff.
Herald added subscribers: cfe-commits, jfb, sunfish, jgravelle-google, sbc100.
We decided to revert this from i64 to i32 in Nov 28 CG meeting. Fixes
PR38632.
Repository:
rC Clang
https://reviews.llvm.org/D51013
Files:
includ
nickdesaulniers created this revision.
nickdesaulniers added a reviewer: aaron.ballman.
Herald added a subscriber: cfe-commits.
Building the Linux kernel with clang is now broken as of commit
cafa0010cd51 ("Raise the minimum required gcc version to 4.6").
We were getting lucky that Clang previous
metzman created this revision.
Herald added a subscriber: cfe-commits.
This allows -fsanitize=fuzzer and -fsanitize=fuzzer-no-link to be used on
Windows.
When either used, Clang will also use some default flags that are needed (such
as incremental linking and debug).
Repository:
rC Clang
ht
saugustine added a comment.
I am refactoring the code that does this, in preparation for adding support for
-shared-libgcc, and, after that, shared compiler-rt. These tests are to defend
against mistakes in that refactoring and prepare for correct handling of the
other cases.
Repository:
rC
saugustine created this revision.
saugustine added a reviewer: echristo.
Herald added a subscriber: cfe-commits.
NFC. Add tests to verify how the driver passes libgcc arguments to the linker
Repository:
rC Clang
https://reviews.llvm.org/D51007
Files:
test/Driver/linux-ld.c
Index: test/Dr
delesley accepted this revision.
delesley added a comment.
This looks good to me. Thanks for the patch.
Comment at: lib/Analysis/ThreadSafety.cpp:932
+ // We're relocking the underlying mutexes. Warn on double locking.
+ if (FSet.findLock(FactMan, UnderCp))
+
jfb accepted this revision.
jfb added inline comments.
Comment at: clang/include/clang/AST/Decl.h:1472
+ /// Do we need to emit an exit-time destructor for this variable?
+ bool isNoDestroy(const ASTContext &) const;
erik.pilkington wrote:
> jfb wrote:
> > rs
NoQ added inline comments.
Comment at: lib/StaticAnalyzer/Checkers/BasicObjCFoundationChecks.cpp:537
+ mutable APIMisuse BT{this, "null passed to CF memory management function"};
+ CallDescription CFRetain{"CFRetain", 1},
+ CFRelease{"CFRelease", 1},
--
aaronpuchert updated this revision to Diff 161598.
aaronpuchert added a comment.
Reformat tests. I promise, this is the last one.
Repository:
rC Clang
https://reviews.llvm.org/D49885
Files:
lib/Analysis/ThreadSafety.cpp
test/SemaCXX/warn-thread-safety-analysis.cpp
Index: test/SemaCXX/wa
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
Seems fine to me, once you add the test for always_destroy + no_destroy.
Comment at: clang/include/clang/Basic/LangOptions.def:311
+LANGOPT(RegisterStaticDestructors, 1, 1,
Author: compnerd
Date: Mon Aug 20 15:50:18 2018
New Revision: 340228
URL: http://llvm.org/viewvc/llvm-project?rev=340228&view=rev
Log:
libclang: add install/distribution targets for python
Add installation support for the python bindings for libclang. Add an
additional CMake configuration variab
aaronpuchert updated this revision to Diff 161596.
aaronpuchert added a comment.
Formatting changes.
Repository:
rC Clang
https://reviews.llvm.org/D49885
Files:
lib/Analysis/ThreadSafety.cpp
test/SemaCXX/warn-thread-safety-analysis.cpp
Index: test/SemaCXX/warn-thread-safety-analysis.cpp
aaronpuchert updated this revision to Diff 161595.
aaronpuchert added a comment.
Proper capitalization of member function, reduction of test code.
Repository:
rC Clang
https://reviews.llvm.org/D49885
Files:
lib/Analysis/ThreadSafety.cpp
test/SemaCXX/warn-thread-safety-analysis.cpp
Index
erik.pilkington added inline comments.
Comment at: clang/include/clang/AST/Decl.h:1472
+ /// Do we need to emit an exit-time destructor for this variable?
+ bool isNoDestroy(const ASTContext &) const;
jfb wrote:
> rsmith wrote:
> > jfb wrote:
> > > This is on
erik.pilkington updated this revision to Diff 161594.
erik.pilkington marked 7 inline comments as done.
erik.pilkington added a comment.
In this new patch:
- Fix some grammar errors in the documentation.
- Add a testcase for both attributes appearing on a variable.
- Just use hasArg() in the driv
This revision was automatically updated to reflect the committed changes.
Closed by commit rCXX340224: Teach libc++ to use native NetBSD's
max_align_t (authored by kamil, committed by ).
Herald added a subscriber: ldionne.
Repository:
rCXX libc++
https://reviews.llvm.org/D47814
Files:
inclu
Author: kamil
Date: Mon Aug 20 15:29:20 2018
New Revision: 340224
URL: http://llvm.org/viewvc/llvm-project?rev=340224&view=rev
Log:
Teach libc++ to use native NetBSD's max_align_t
Summary:
The NetBSD headers ship with max_align_t, that is not
compatible with the fallback version in libc++.
There
This revision was automatically updated to reflect the committed changes.
Closed by commit rC340220: [ASTImporter] Add test for C++'s try/catch
statements. (authored by teemperor, committed by ).
Repository:
rC Clang
https://reviews.llvm.org/D50978
Files:
test/Import/cxx-try-catch/Inputs/F.
Author: teemperor
Date: Mon Aug 20 15:13:24 2018
New Revision: 340220
URL: http://llvm.org/viewvc/llvm-project?rev=340220&view=rev
Log:
[ASTImporter] Add test for C++'s try/catch statements.
Summary: Also enable exceptions in clang-import-test so that we can parse the
test files.
Reviewers: a.s
teemperor updated this revision to Diff 161580.
teemperor added a comment.
- Clarified that the Exception options in LangOpts are not related to the
dynamic_cast support by moving them above the comment (Thanks Aleksei!)
https://reviews.llvm.org/D50978
Files:
test/Import/cxx-try-catch/Inputs
teemperor created this revision.
Herald added subscribers: cfe-commits, martong.
Herald added a reviewer: a.sidorin.
Repository:
rC Clang
https://reviews.llvm.org/D51001
Files:
test/Import/cxx-for-range/Inputs/F.cpp
test/Import/cxx-for-range/test.cpp
Index: test/Import/cxx-for-range/test
rsmith added inline comments.
Comment at: lib/Basic/SourceLocation.cpp:89-93
+ B.print(OS, SM);
+ OS << ",\n ";
+ E.print(OS, SM);
+ OS << "]\n";
+}
It would seem somewhat more natural to dump `SourceRange`s the same way we dump
them in `-ast-dump`. (That is
a_sidorin requested changes to this revision.
a_sidorin added a comment.
This revision now requires changes to proceed.
Hello Stephen,
These methods will be really useful.
Comment at: lib/Basic/SourceLocation.cpp:90
+ B.print(OS, SM);
+ OS << ",\n ";
+ E.print(OS, SM);
-
steveire added a comment.
Thanks, I've done that now.
Repository:
rC Clang
https://reviews.llvm.org/D50662
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
steveire updated this revision to Diff 161570.
steveire added a comment.
Add dump() and supporting methods to SourceRange
Repository:
rC Clang
https://reviews.llvm.org/D50662
Files:
include/clang/Basic/SourceLocation.h
lib/Basic/SourceLocation.cpp
Index: lib/Basic/SourceLocation.cpp
==
Author: rsmith
Date: Mon Aug 20 14:47:29 2018
New Revision: 340215
URL: http://llvm.org/viewvc/llvm-project?rev=340215&view=rev
Log:
Model type attributes as regular Attrs.
Specifically, AttributedType now tracks a regular attr::Kind rather than
having its own parallel Kind enumeration, and Attri
jfb added inline comments.
Comment at: clang/include/clang/AST/Decl.h:1472
+ /// Do we need to emit an exit-time destructor for this variable?
+ bool isNoDestroy(const ASTContext &) const;
rsmith wrote:
> jfb wrote:
> > This is only valid for static variables
rsmith added inline comments.
Comment at: clang/include/clang/AST/Decl.h:1472
+ /// Do we need to emit an exit-time destructor for this variable?
+ bool isNoDestroy(const ASTContext &) const;
jfb wrote:
> This is only valid for static variables, right? It's p
kadircet added inline comments.
Comment at: unittests/clangd/CodeCompleteTests.cpp:1710
+ $bol^
+ ab$ab^
+ x.ab$dot^
Maybe an EXPECT for this one as well?
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D50962
__
rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
In https://reviews.llvm.org/D50527#1206460, @erik.pilkington wrote:
> Ping!
If the build came back clean, then I think our combination of previous
sign-offs is good enough. :)
https://
beanz added a comment.
Just want to bring an IRC conversation that I had with @phosek into the proper
code review.
This patch is great as-is, but if we want to extend this to Darwin there are
some problems because of how Darwin handles triples. Specifically Darwin has
multiple potentially vali
jfb added inline comments.
Comment at: clang/include/clang/AST/Decl.h:1472
+ /// Do we need to emit an exit-time destructor for this variable?
+ bool isNoDestroy(const ASTContext &) const;
This is only valid for static variables, right? It's probably better t
a_sidorin accepted this revision.
a_sidorin added inline comments.
This revision is now accepted and ready to land.
Comment at: tools/clang-import-test/clang-import-test.cpp:199
Inv->getLangOpts()->RTTI = true;
+ Inv->getLangOpts()->Exceptions = true;
+ Inv->getLangOpts()->C
george.karpenkov added inline comments.
Comment at: lib/StaticAnalyzer/Checkers/BasicObjCFoundationChecks.cpp:537
+ mutable APIMisuse BT{this, "null passed to CF memory management function"};
+ CallDescription CFRetain{"CFRetain", 1},
+ CFRelease{"CFRelease", 1
krytarowski added a comment.
@chandlerc thank you for your note. I've originally treated this change as an
obvious code change, just the reviewers regardless of reviewing an analogous
change for MUSL, had no interest in NetBSD.
Repository:
rL LLVM
https://reviews.llvm.org/D47814
erik.pilkington added a comment.
Ping!
https://reviews.llvm.org/D50527
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
erik.pilkington created this revision.
erik.pilkington added reviewers: jfb, rsmith, aaron.ballman, rjmccall, bruno.
Herald added a subscriber: dexonsmith.
See the recent thread here:
http://lists.llvm.org/pipermail/cfe-dev/2018-July/058494.html
This patch adds the flag -fno-c++-static-destructo
chandlerc added a comment.
In https://reviews.llvm.org/D47814#1206372, @krytarowski wrote:
> If there are no more comments, I will land this by the end of this week.
Just for the record, this is not OK and not how LLVM's code review works.
You can and must wait for review. I think Joerg alread
eandrews marked an inline comment as done.
eandrews added a comment.
In https://reviews.llvm.org/D40925#1206416, @rnk wrote:
> lgtm!
Thanks!
https://reviews.llvm.org/D40925
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm
Author: dblaikie
Date: Mon Aug 20 13:14:08 2018
New Revision: 340206
URL: http://llvm.org/viewvc/llvm-project?rev=340206&view=rev
Log:
DebugInfo: Add the ability to disable DWARF name tables entirely
This changes the current default behavior (from emitting pubnames by
default, to not emitting the
Dmitry.Kozhevnikov created this revision.
Dmitry.Kozhevnikov added reviewers: ilya-biryukov, sammccall.
Herald added subscribers: cfe-commits, kadircet, arphaman, jkorous, MaskRay,
ioeric.
By default it's 512K, which is way to small for clang parser to run on. There
is no way to do it via platfo
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm!
https://reviews.llvm.org/D40925
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
hugoeg updated this revision to Diff 161550.
hugoeg added a comment.
refined test to include some more cases as a result of the new matcher
https://reviews.llvm.org/D50542
Files:
clang-tidy/abseil/AbseilTidyModule.cpp
clang-tidy/abseil/CMakeLists.txt
clang-tidy/abseil/NoInternalDepsCheck.
eandrews marked an inline comment as done.
eandrews added inline comments.
Comment at: include/clang/Basic/LangOptions.def:311
+BENIGN_LANGOPT(KeepStaticConsts , 1, 0, "keep static const variables even
if unused")
+
rnk wrote:
> Let's make this a CodeGenO
mstorsjo added a comment.
In https://reviews.llvm.org/D50564#1206370, @cdavis5x wrote:
> In https://reviews.llvm.org/D50564#1206302, @kristina wrote:
>
> > I'm all for this change except the core issue is that you're using
> > libunwind as a shim around the actual unwinding API provided by Windo
eandrews updated this revision to Diff 161544.
eandrews added a comment.
Based on Reid's feedback, I changed option to CodeGenOption
https://reviews.llvm.org/D40925
Files:
include/clang/Driver/Options.td
include/clang/Frontend/CodeGenOptions.def
lib/CodeGen/CodeGenModule.cpp
lib/Driver/
krytarowski added a comment.
If there are no more comments, I will land this by the end of this week.
Repository:
rL LLVM
https://reviews.llvm.org/D47814
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailma
cdavis5x added a comment.
In https://reviews.llvm.org/D50564#1206302, @kristina wrote:
> I'm all for this change except the core issue is that you're using libunwind
> as a shim around the actual unwinding API provided by Windows. It would be
> nice to have something that did not have to do tha
rjmccall added a comment.
In https://reviews.llvm.org/D50616#1206181, @leonardchan wrote:
> I made a post on llvm-dev
> (http://lists.llvm.org/pipermail/llvm-dev/2018-August/125433.html) to see if
> other people have opinions on this. In the meantime, do you think I should
> make a separate pa
vsk updated this revision to Diff 161536.
vsk added a comment.
- Here is a GIF that might help illustrate the bug:
http://net.vedantk.com/static/llvm/lambda-implicit-capture-bug.gif
- Update test/SemaCXX/uninitialized.cpp to highlight the behavior change which
comes from using getBeginOrDeclLoc(
ioeric updated this revision to Diff 161537.
ioeric added a comment.
- Make sure completion result callback can be called even if the unused
speculative request has not finished.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D50962
Files:
clangd/ClangdServer.cpp
clangd/Cla
t.p.northover added a comment.
> EmitAArch64BuiltinExpr() also emits args into Ops before the big switch (with
> some more subtlety around the last arg that I don't understand), but then
> almost every switch case does EmitScalarExpr(E->getArg(n)).
Took me a while to remember, but I can at leas
kbarton created this revision.
kbarton added reviewers: power-llvm-team, hfinkel, echristo, rsmith.
Herald added a subscriber: nemanjai.
This is the clang counterpart to the patch posted in
https://reviews.llvm.org/D50988.
The patch removes Darwin support from the POWER backend. A similar approa
kristina added a comment.
I'm all for this change except the core issue is that you're using libunwind as
a shim around the actual unwinding API provided by Windows. It would be nice to
have something that did not have to do that and was capable of performing
unwinding of SEH-style exceptions w
krytarowski added a comment.
From my point of view, the number of people who have idea about these types and
why do they differ between ABIs and OSes is so low, that it's even hard to get
someone to review it here.
Repository:
rUNW libunwind
https://reviews.llvm.org/D50413
__
This revision was automatically updated to reflect the committed changes.
Closed by commit rC340198: [Lex] Make HeaderMaps a unique_ptr vector (authored
by MaskRay, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D50945?vs=161391&id=161530#toc
Repository:
rC Clang
https://
MaskRay added a comment.
Thanks!
Repository:
rC Clang
https://reviews.llvm.org/D50945
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: maskray
Date: Mon Aug 20 12:15:02 2018
New Revision: 340198
URL: http://llvm.org/viewvc/llvm-project?rev=340198&view=rev
Log:
[Lex] Make HeaderMaps a unique_ptr vector
Summary: unique_ptr makes the ownership clearer than a raw pointer container.
Reviewers: Eugene.Zelenko, dblaikie
Subsc
kristina added a comment.
This doesn't seem like a great idea to me, it's a negligible amount of type
safety gained over using `stdint.h` types most people are more familiar with
anyway. If you need something niche like those typedefs it's probably better to
include the header and define them y
thakis added a comment.
In https://reviews.llvm.org/D50979#1206282, @thakis wrote:
> In https://reviews.llvm.org/D50979#1206211, @rsmith wrote:
>
> > I don't think this is NFC. Testcase:
> >
> > long long int a, b, c, d;
> > unsigned char f() { return _InterlockedCompareExchange128(&(++a), ++
deannagarcia added inline comments.
Comment at: clang-tidy/abseil/NoNamespaceCheck.cpp:23
+
+ Finder->addMatcher(namespaceDecl(hasName("absl")).bind("absl_namespace"),
+ this);
hokein wrote:
> JonasToth wrote:
> > hugoeg wrote:
> > > deannaga
thakis added a comment.
In https://reviews.llvm.org/D50979#1206211, @rsmith wrote:
> I don't think this is NFC. Testcase:
>
> long long int a, b, c, d;
> unsigned char f() { return _InterlockedCompareExchange128(&(++a), ++b, ++c,
> &(++d)); }
>
>
> Today, Clang increments `a`, `b`, `c`, and
deannagarcia updated this revision to Diff 161526.
deannagarcia edited the summary of this revision.
deannagarcia added a comment.
This revision includes a matcher so that the warning does not trigger on
internal Abseil files.
https://reviews.llvm.org/D50580
Files:
clang-tidy/abseil/AbseilMa
joerg added a comment.
Is there a reason for defining them? As in: does anything outside libunwind use
them? I haven't seen such software yet.
Repository:
rUNW libunwind
https://reviews.llvm.org/D50413
___
cfe-commits mailing list
cfe-commits@li
rsmith added a comment.
In https://reviews.llvm.org/D49511#1206265, @rsmith wrote:
> In https://reviews.llvm.org/D49511#1194716, @leonardchan wrote:
>
> > @rsmith any more feedback on this current version? If it still looks
> > incorrect to use the record this way, I don't mind simplifying it to
rsmith added a comment.
In https://reviews.llvm.org/D49511#1194716, @leonardchan wrote:
> @rsmith any more feedback on this current version? If it still looks
> incorrect to use the record this way, I don't mind simplifying it to work on
> lvalue to rvalue conversions without checking for a lea
cdavis5x added a comment.
Ping...
Repository:
rUNW libunwind
https://reviews.llvm.org/D50564
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cdavis5x added a comment.
Ping.
Are y'all waiting for me to do something?
Repository:
rUNW libunwind
https://reviews.llvm.org/D50413
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi
arsenm created this revision.
arsenm added reviewers: rampitec, kzhuravl, yaxunl.
Herald added subscribers: t-tye, tpr, dstuttard, nhaehnle, wdng, jvesely.
arsenm added a dependency: D50983: AMDGPU: Partially move target handling code
from clang to TargetParser.
https://reviews.llvm.org/D50984
F
rsmith added a comment.
I don't think this is NFC. Testcase:
long long int a, b, c, d;
unsigned char f() { return _InterlockedCompareExchange128(&(++a), ++b, ++c,
&(++d)); }
Today, Clang increments `a`, `b`, `c`, and `d` twice each in `f()`.
https://reviews.llvm.org/D50979
vsapsai added a comment.
Herald added a subscriber: kadircet.
Thanks, Dmitry, for your explanation. It does help to understand your use case
better.
What about having a mode that treats missing header as non-fatal error? Because
I believe there are cases where there is no sense to continue afte
arsenm closed this revision.
arsenm added a comment.
r340193
https://reviews.llvm.org/D50957
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: arsenm
Date: Mon Aug 20 11:16:48 2018
New Revision: 340193
URL: http://llvm.org/viewvc/llvm-project?rev=340193&view=rev
Log:
Rename -mlink-cuda-bitcode to -mlink-builtin-bitcode
The same semantics work for OpenCL, and probably any offload
language. Keep the old name around as an alias.
M
leonardchan added a comment.
I made a post on llvm-dev
(http://lists.llvm.org/pipermail/llvm-dev/2018-August/125433.html) to see if
other people have opinions on this. In the meantime, do you think I should make
a separate patch that moves this into an LLVM intrinsic function?
Repository:
r
pirama abandoned this revision.
pirama added a comment.
Thanks for the clarification Richard and Eli. I agree that leaving the status
quo will match the intent of the macro. I'll abandon this.
Repository:
rC Clang
https://reviews.llvm.org/D50683
_
Author: abataev
Date: Mon Aug 20 11:03:40 2018
New Revision: 340191
URL: http://llvm.org/viewvc/llvm-project?rev=340191&view=rev
Log:
[OPENMP] Fix crash on the emission of the weak function declaration.
If the function is actually a weak reference, it should not be marked as
deferred definition a
deannagarcia updated this revision to Diff 161513.
https://reviews.llvm.org/D50862
Files:
clang-tidy/abseil/AbseilTidyModule.cpp
clang-tidy/abseil/CMakeLists.txt
clang-tidy/abseil/FasterStrsplitDelimiterCheck.cpp
clang-tidy/abseil/FasterStrsplitDelimiterCheck.h
docs/ReleaseNotes.rst
d
dblaikie accepted this revision.
dblaikie added a comment.
This revision is now accepted and ready to land.
Looks good - thanks!
Repository:
rC Clang
https://reviews.llvm.org/D50945
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://l
dblaikie added a comment.
In https://reviews.llvm.org/D50945#1205337, @kristina wrote:
> Given the context (class an file name itself) and documentation around the
> function, I don't think in this particular case it improves readability or
> maintainability, the lifetime of the `HeaderMap` is
thakis created this revision.
thakis added a reviewer: rnk.
Herald added a reviewer: javed.absar.
Herald added a subscriber: kristof.beyls.
EmitX86BuiltinExpr() emits all args into Ops at the beginning, so don't do that
work again. No intended behavior change.
(TNorthover: EmitAArch64BuiltinExpr
jolesiak added a comment.
Nice! That looks very promising.
It still fails when we pass a longer range (maybe mimicking `PreviousRBrace`
will help), e.g.:
Base code:
@protocol A
@optional
// comment
- (void)f;
@end
MACRO
formatted with `clang-format -lines=3:6 file.m` gives:
@pr
arphaman updated this revision to Diff 161504.
arphaman marked 2 inline comments as done.
arphaman added a comment.
Address review comments.
Repository:
rC Clang
https://reviews.llvm.org/D50926
Files:
include/clang/Basic/SourceManager.h
lib/Basic/SourceManager.cpp
unittests/Basic/Sourc
arphaman added inline comments.
Comment at: include/clang/Basic/SourceManager.h:1539
+ /// \returns true if the callback returned true, false otherwise.
+ bool findFileIDsForFile(const FileEntry *SourceFile,
+ llvm::function_ref Callback) const;
---
vsk added subscribers: jkorous, vsapsai.
vsk added a comment.
+ Jan and Volodymyr. This seemed to be in good shape the last time I looked at
it. Not having touched libclang for a while I don't think I can give an
official lgtm.
Repository:
rC Clang
https://reviews.llvm.org/D42043
___
teemperor created this revision.
Herald added subscribers: cfe-commits, martong.
Herald added a reviewer: a.sidorin.
Also enable exceptions in clang-import-test so that we can parse the test files.
Repository:
rC Clang
https://reviews.llvm.org/D50978
Files:
test/Import/cxx-try-catch/Inputs
This revision was automatically updated to reflect the committed changes.
Closed by commit rL340188: Close FileEntries of cached files in
ModuleManager::addModule(). (authored by adrian, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D50870?vs=161119&id=161501#toc
Repository
Author: adrian
Date: Mon Aug 20 10:10:27 2018
New Revision: 340188
URL: http://llvm.org/viewvc/llvm-project?rev=340188&view=rev
Log:
Close FileEntries of cached files in ModuleManager::addModule().
While investigating why LLDB (which can build hundreds of clang
modules during one debug session) w
rsmith added inline comments.
Comment at: lib/Parse/ParseExpr.cpp:1126
+
+Actions.StartCheckingNoDeref();
+
This parser-driven start/stop mechanism will not work in C++ templates.
Instead, can you remove the "start" part and check the noderef exprs as part o
hugoeg marked an inline comment as done.
hugoeg added inline comments.
Comment at: clang-tidy/abseil/NoInternalDepsCheck.cpp:20
+
+bool IsInAbseilFile(const SourceManager& manager, SourceLocation loc){
+ if (loc.isInvalid()) {
hokein wrote:
> hugoeg wrote:
> > h
arphaman added a comment.
In https://reviews.llvm.org/D50571#1205650, @joaotavora wrote:
> > LGTM. Let's watch out for possible breakages in any of the clients, though.
> > I've checked VSCode and it seems to be fine with the added fields.
>
> This isn't in the spec and broke the LSP client eglo
1 - 100 of 177 matches
Mail list logo
