Author: kadircet
Date: Tue Oct 9 03:29:54 2018
New Revision: 344033
URL: http://llvm.org/viewvc/llvm-project?rev=344033&view=rev
Log:
[clangd] Mark colon as a safe character when percent-encoding.
Summary: Also change output of percent-encoding to use upper-case letters.
Reviewers: sammccall
R
alexshap added a comment.
@grimar, this is an interesting observation which I've had on my mind for quite
some time as well; a couple of things which I have not double-checked yet -
just in case - do both gold and lld completely ignore dwo-related sections ?
(did you check that ?), and another
hokein created this revision.
hokein added a reviewer: sammccall.
Herald added subscribers: kadircet, arphaman, jkorous, MaskRay, ioeric,
ilya-biryukov.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D53019
Files:
clangd/index/dex/dexp/Dexp.cpp
Index: clangd/index/dex/dexp/De
Szelethus updated this revision to Diff 168775.
Szelethus added a comment.
This revision is now accepted and ready to land.
Finally managed to run creduce. I added the test that caused a crash on our
server, but as I said, I couldn't replicate it elsewhere.
https://reviews.llvm.org/D51300
File
Szelethus added a comment.
Since a good amount of time passed since this patch was made, I'll re-analyze
the LLVM project with this patch rebased on trunk just to be sure that nothing
breaks.
https://reviews.llvm.org/D51300
___
cfe-commits mailing
olista01 added inline comments.
Comment at: lib/Frontend/CompilerInvocation.cpp:1133
+ if (Arg *A = Args.getLastArg(OPT_msign_return_address_EQ)) {
+const auto SignScopeKey = StringRef(A->getValue()).split('+');
+StringRef SignScope = SignScopeKey.first;
MTC added a comment.
Greate idea! If we can enrich this list, it will not only help the reviewer,
but also help beginners, like me, avoid some pitfalls when developing a new
checker.
I agree with NoQ to classify the lists. In addition to the two categories
proposed by NoQ, there is another cla
baloghadamsoftware updated this revision to Diff 168787.
baloghadamsoftware added a comment.
Updated according to the comments.
https://reviews.llvm.org/D52727
Files:
clang-tidy/performance/ForRangeCopyCheck.cpp
clang-tidy/performance/ForRangeCopyCheck.h
clang-tidy/performance/Unnecessary
baloghadamsoftware marked 2 inline comments as done.
baloghadamsoftware added inline comments.
Comment at: clang-tidy/performance/UnnecessaryCopyInitialization.cpp:65-66
has(varDecl(hasLocalStorage(),
- hasType(matchers::i
sammccall added inline comments.
Comment at: clangd/index/dex/dexp/Dexp.cpp:172
+ "filter", cl::init(".*"),
+ cl::desc("Print all results from files matching this filter."),
+ };
filter -> regular expression
Comment at: clangd/index/
Author: jonastoth
Date: Tue Oct 9 06:24:50 2018
New Revision: 344044
URL: http://llvm.org/viewvc/llvm-project?rev=344044&view=rev
Log:
[clangd] fix miscompiling lower_bound call
Modified:
clang-tools-extra/trunk/clangd/index/Index.cpp
Modified: clang-tools-extra/trunk/clangd/index/Index.cpp
Author: jonastoth
Date: Tue Oct 9 06:29:31 2018
New Revision: 344046
URL: http://llvm.org/viewvc/llvm-project?rev=344046&view=rev
Log:
[clang-tidy] NFC fix warnings from missing braces
The std::array create multiple StringRef but did not wrap
them in braces. Some compilers warned for that. Addin
JonasToth added inline comments.
Comment at: clang-tidy/performance/UnnecessaryCopyInitialization.cpp:65-66
has(varDecl(hasLocalStorage(),
- hasType(matchers::isExpensiveToCopy()),
+ hasTy
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM!
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D52859
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
jranieri-grammatech added a comment.
In https://reviews.llvm.org/D52905#1257040, @NoQ wrote:
> Hmmm, interesting. A checker doesn't usually need to access these specific
> static locals, at least not directly. These are usually accessed through
> functions in .cpp files that are supposed to be
aaron.ballman added a comment.
I'm not all that keen on the idea of deprecating `set output` like this -- that
command has been around since forever, and scripts outside of our control are
likely to rely on it. However, it seems like you should be able to make `set
output` accept a comma-delimi
joey created this revision.
joey added reviewers: asavonic, Anastasia.
Herald added subscribers: cfe-commits, yaxunl, mgorny.
This is the prototype for the approach that was mentioned by Anastasia in
http://lists.llvm.org/pipermail/cfe-dev/2018-September/059529.html
The tablegen file describes t
hans added inline comments.
Comment at: clang/lib/Sema/SemaDeclCXX.cpp:5599
+bool Sema::isInlineFunctionDLLExportable(const CXXMethodDecl *MD) {
+ assert(MD->isInlined());
Okay, breaking out this logic is a little better, but I still don't like that
we now ha
Author: abataev
Date: Tue Oct 9 07:49:00 2018
New Revision: 344049
URL: http://llvm.org/viewvc/llvm-project?rev=344049&view=rev
Log:
[OPENMP][NVPTX] Support memory coalescing for globalized variables.
Added support for memory coalescing for better performance for
globalized variables. From now o
jyknight accepted this revision.
jyknight added a comment.
This revision is now accepted and ready to land.
Given that there's no technical reason for the compiler to prohibit this (it
was just clang trying to diagnose a probable user-error, which turns out to not
be as probable as ), this seems
steveire added a comment.
- The scripts will continue to work at least until `set output` is removed,
which is not going to happen soon.
- A comma-delimited list of options means that if I have `foo, bar, bat`
enabled and want to add `bang`, I need to `set output foo, bar, bat, bang`. Or
alter
hokein updated this revision to Diff 168801.
hokein marked 3 inline comments as done.
hokein added a comment.
Herald added a subscriber: mgorny.
Address review comments:
- provide query by qualified name (with -name)
- add -name support for Lookup.
Repository:
rCTE Clang Tools Extra
https://
hokein added inline comments.
Comment at: clangd/index/dex/dexp/Dexp.cpp:204
{"lookup", "Dump symbol details by ID", llvm::make_unique},
+{"refs", "Find references by qualified name", llvm::make_unique},
};
sammccall wrote:
> I'm not sure "by qualified
hokein updated this revision to Diff 168802.
hokein added a comment.
Minor fix.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D53019
Files:
clangd/index/dex/dexp/CMakeLists.txt
clangd/index/dex/dexp/Dexp.cpp
Index: clangd/index/dex/dexp/Dexp.cpp
===
sammccall added inline comments.
Comment at: clangd/index/dex/dexp/Dexp.cpp:65
+ clang::clangd::SymbolID SymID;
+ // We choose the first one if there are overloaded symbols.
+ Index->fuzzyFind(Request, [&](const Symbol &Sym) {
I don't think this is reasonable
Author: hokein
Date: Tue Oct 9 08:16:14 2018
New Revision: 344054
URL: http://llvm.org/viewvc/llvm-project?rev=344054&view=rev
Log:
[clangd] Fix an accident change in r342999.
Modified:
clang-tools-extra/trunk/clangd/index/YAMLSerialization.cpp
Modified: clang-tools-extra/trunk/clangd/index
Author: ioeric
Date: Tue Oct 9 08:17:16 2018
New Revision: 344055
URL: http://llvm.org/viewvc/llvm-project?rev=344055&view=rev
Log:
[clang-move] Fix broken json output.
Modified:
clang-tools-extra/trunk/clang-move/tool/ClangMoveMain.cpp
Modified: clang-tools-extra/trunk/clang-move/tool/Clan
yaxunl added inline comments.
Comment at: lib/Sema/SemaDeclCXX.cpp:7231
+if (ICI)
+ CSM = getSpecialMember(MD);
+
jlebar wrote:
> LGTM, but perhaps we should use a new variable instead of modifying `CSM` in
> case someone adds code beneath this branch?
Szelethus created this revision.
Szelethus added reviewers: NoQ, george.karpenkov, xazax.hun, MTC.
Herald added subscribers: cfe-commits, jfb, mikhail.ramalho, a.sidorin,
rnkovacs, szepet, whisperity.
Added some extra tasks to the open projects. These are the ideas of @NoQ and
@george.karpenkov,
Szelethus added a comment.
Please reupload with full context (`-U`).
Comment at: lib/StaticAnalyzer/Core/Checker.cpp:20
+int ImplicitNullDerefEvent::Tag;
+
nit: Static fields initialize to 0 without out of line definition.
Repository:
rC Clang
https:
Szelethus added inline comments.
Comment at: lib/StaticAnalyzer/Core/Checker.cpp:20
+int ImplicitNullDerefEvent::Tag;
+
Szelethus wrote:
> nit: Static fields initialize to 0 without out of line definition.
Never mind, you still have to define it. It's been a wh
Szelethus added a comment.
Please reupload with full context (`-U9`).
Repository:
rC Clang
https://reviews.llvm.org/D52905
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: yaxunl
Date: Tue Oct 9 08:53:14 2018
New Revision: 344057
URL: http://llvm.org/viewvc/llvm-project?rev=344057&view=rev
Log:
[CUDA][HIP] Fix ShouldDeleteSpecialMember for inherited constructors
ShouldDeleteSpecialMember is called upon inherited constructors.
It calls inferCUDATargetForImp
This revision was automatically updated to reflect the committed changes.
Closed by commit rL344057: [CUDA][HIP] Fix ShouldDeleteSpecialMember for
inherited constructors (authored by yaxunl, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.o
alexfh added inline comments.
Comment at: clang-tidy/modernize/MakeSmartPtrCheck.cpp:246
+ ASTContext *Ctx) {
+ std::function SkipParensParents =
+ [&](const Expr *E) {
hokein wrote:
> nit: I'd use `auto` to avoid this long
alexfh updated this revision to Diff 168812.
alexfh marked 2 inline comments as done.
alexfh added a comment.
- Remove recursion, use `auto`.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D52989
Files:
clang-tidy/modernize/MakeSmartPtrCheck.cpp
clang-tidy/modernize/MakeSmar
Author: alexfh
Date: Tue Oct 9 08:58:18 2018
New Revision: 344058
URL: http://llvm.org/viewvc/llvm-project?rev=344058&view=rev
Log:
[clang-tidy] Fix handling of parens around new expressions in make_
checks.
Summary:
Extra parentheses around a new expression result in incorrect code
after apply
This revision was automatically updated to reflect the committed changes.
Closed by commit rL344058: [clang-tidy] Fix handling of parens around new
expressions in make_… (authored by alexfh, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews.
eandrews added a comment.
Yes. I understand. At the moment, exception handling flags are generated based
on `BENCHMARK_ENABLE_EXCEPTIONS` in `utils/benchmark/CMakeLists.txt` .
However, `_HAS_EXCEPTIONS` is not defined based on this (code below). The
warnings are a result of this mismatch.
jranieri-grammatech updated this revision to Diff 168815.
jranieri-grammatech added a comment.
Added more context.
Repository:
rC Clang
https://reviews.llvm.org/D52905
Files:
include/clang/StaticAnalyzer/Core/PathSensitive/DynamicTypeMap.h
include/clang/StaticAnalyzer/Core/PathSensitive/
kpn updated this revision to Diff 168814.
kpn added a comment.
Update based on feedback to https://reviews.llvm.org/D52839: add missing AST
(de)serialization support.
Ping.
https://reviews.llvm.org/D51372
Files:
include/clang/AST/Expr.h
include/clang/AST/ExprCXX.h
lib/AST/ASTImporter.cp
Anastasia accepted this revision.
Anastasia added a comment.
This revision is now accepted and ready to land.
LGTM! Thanks!
https://reviews.llvm.org/D52292
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman
Anastasia added a comment.
In https://reviews.llvm.org/D51402#1252619, @sidorovd wrote:
> @Anastasia , since there is a problem I described, wouldn't you mind if I
> stay with the first iteration of the patch (adding the extension to
> OpenCLExtensions.def) and after we'll investigate what is w
Szelethus added a comment.
Mind you, there are some ideas I didn't add to the list -- I just don't know
how to put them to words nicely, but I'm on it.
Also, a lot of these is outdated, but I joined the project relatively recently,
so I'm not sure what's the state on all of them.
Repository:
Szelethus added inline comments.
Comment at: www/analyzer/alpha_checks.html:352-405
+
+alpha.cplusplus.InvalidatedIterator
+(C++)
+Check for use of invalidated iterators.
+
+
@baloghadamsoftware Is this a good description of your checker(s)?
https://reviews.l
ymandel created this revision.
ymandel added reviewers: aaron.ballman, JonasToth, ioeric.
Herald added subscribers: cfe-commits, xazax.hun, mgorny.
Adds a new check readability-const-value-return, which checks for functions with
a ``const``-qualified return type and recommends removal of the `cons
Eugene.Zelenko added a comment.
GCC has -Wignored-qualifiers for long time, so may be better to implement it in
Clang?
Comment at: clang-tidy/readability/ConstValueReturnCheck.cpp:66
+ llvm::Optional Loc = findConstToRemove(Def, Result);
+ if (!Loc) return;
+ DiagnosticBuil
ashi1 updated this revision to Diff 168836.
ashi1 added a comment.
The device libraries has been updated, and the hip.amdgcn.bc library is now
available there.
Repository:
rC Clang
https://reviews.llvm.org/D52673
Files:
lib/Driver/ToolChains/HIP.cpp
test/Driver/hip-device-libs.hip
Ind
yaxunl added inline comments.
Comment at: lib/Driver/ToolChains/HIP.cpp:85
C.addTempFile(C.getArgs().MakeArgString(TmpName));
CmdArgs.push_back(OutputFileName);
SmallString<128> ExecPath(C.getDriver().Dir);
maybe we should put hip.amdgcn.bc at the be
Anastasia added a comment.
It would be good to test your `CIndex` changes in `test/Index/opencl-types.cl`.
Comment at: test/SemaOpenCL/intel-subgroup-avc-ext-types.cl:26
+ char4 c4, event_t e, struct st ss) {
+ intel_sub_group_avc_mce_payload_t payload_mce = 0; // No z
Author: rsmith
Date: Tue Oct 9 11:49:22 2018
New Revision: 344070
URL: http://llvm.org/viewvc/llvm-project?rev=344070&view=rev
Log:
PR39231: fix null dereference when diagnosing deduction failure due to
conflicting values for a non-type pack.
Modified:
cfe/trunk/lib/Sema/SemaOverload.cpp
sammccall created this revision.
sammccall added a reviewer: ioeric.
Herald added subscribers: cfe-commits, jfb, kadircet, arphaman, jkorous,
MaskRay, ilya-biryukov, mgorny.
See tinyurl.com/clangd-automatic-index for design and goals.
Lots of limitations to keep this patch smallish, TODOs everyw
mstorsjo updated this revision to Diff 168848.
mstorsjo added a comment.
Relying on a linker pragma in sanitizers and mingw lib name logic in lld.
https://reviews.llvm.org/D52990
Files:
lib/CodeGen/TargetInfo.cpp
test/Driver/fsanitize.c
Index: test/Driver/fsanitize.c
=
mstorsjo added a comment.
If relying on logic in lld, this one can be discarded.
Repository:
rC Clang
https://reviews.llvm.org/D53013
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi
sammccall created this revision.
sammccall added reviewers: ioeric, ilya-biryukov.
Herald added subscribers: cfe-commits, kadircet, arphaman, jkorous, MaskRay.
I think this was just copied from somewhere with the belief that it actually
did some crash handling.
Of course the question arises: *sho
SGTM.
On Tue, Oct 9, 2018, 7:57 AM James Y Knight via Phabricator <
revi...@reviews.llvm.org> wrote:
> jyknight accepted this revision.
> jyknight added a comment.
> This revision is now accepted and ready to land.
>
> Given that there's no technical reason for the compiler to prohibit this
> (it
ashi1 updated this revision to Diff 168855.
ashi1 marked an inline comment as done.
ashi1 added a comment.
I've moved the hip.amdgcn.bc to the top of the libs. Updated test as well.
https://reviews.llvm.org/D52673
Files:
lib/Driver/ToolChains/HIP.cpp
test/Driver/hip-device-libs.hip
Index:
ymandel updated this revision to Diff 168856.
ymandel added a comment.
Minor changes addressing comments.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D53025
Files:
clang-tidy/readability/CMakeLists.txt
clang-tidy/readability/ConstValueReturnCheck.cpp
clang-tidy/readabil
ymandel updated this revision to Diff 168857.
ymandel marked 2 inline comments as done.
ymandel added a comment.
Dropped unneeded clang qualifier.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D53025
Files:
clang-tidy/readability/CMakeLists.txt
clang-tidy/readability/ConstV
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
Cool, thanks! I'll commit.
Repository:
rC Clang
https://reviews.llvm.org/D52905
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists
NoQ added a comment.
> where `isStaticLocal` is defined as:
You can send this one as well if you like! It's weird that we don't already
have it.
Repository:
rC Clang
https://reviews.llvm.org/D52905
___
cfe-commits mailing list
cfe-commits@lists
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
Yup, this one looks great as well. I'll commit. Thanks!
Repository:
rC Clang
https://reviews.llvm.org/D52906
___
cfe-commits mailing list
cfe-commit
ymandel updated this revision to Diff 168858.
ymandel added a comment.
Add missing const.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D53025
Files:
clang-tidy/readability/CMakeLists.txt
clang-tidy/readability/ConstValueReturnCheck.cpp
clang-tidy/readability/ConstValueRe
NoQ added a comment.
In https://reviews.llvm.org/D52905#1259249, @NoQ wrote:
> > where `isStaticLocal` is defined as:
>
> You can send this one as well if you like! It's weird that we don't already
> have it.
Or actually maybe it can be implemented as `allOf(hasStaticStorageDuration(),
unless
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
Thanks, looks like this has been largely redundant since r294800 removed the
only interesting thing that `EM_OffsetFold` did.
https://reviews.llvm.org/D52924
__
ymandel marked 3 inline comments as done.
ymandel added a comment.
Thanks for the review. I've addressed the comments.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D53025
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http
rsmith added inline comments.
Comment at: clang/lib/AST/ExprConstant.cpp:12-32
// Constant expression evaluation produces four main results:
//
// * A success/failure flag indicating whether constant folding was
successful.
//This is the 'bool' return value used by most
NoQ added a comment.
In https://reviews.llvm.org/D52983#1258499, @xazax.hun wrote:
> In https://reviews.llvm.org/D52983#1258466, @NoQ wrote:
>
> > Yay, these look useful. Is there also an attribute for methods that should
> > never be called on a 'moved-from' object?
>
>
> I do not know about su
NoQ added a comment.
Whoa thanks! Will have a closer look again.
Comment at: www/analyzer/open_projects.html:33
+
+Handle aggregate construction.
+Aggregates are object that can be brace-initialized without calling a
I'll try to list other constructor k
aaron.ballman added a comment.
In https://reviews.llvm.org/D52857#1258893, @steveire wrote:
> - The scripts will continue to work at least until `set output` is removed,
> which is not going to happen soon.
Certainly, but given that deprecation implies eventual removal, people are
still being
george.burgess.iv accepted this revision.
george.burgess.iv added a comment.
Thanks!
https://reviews.llvm.org/D52924
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sidneym created this revision.
sidneym added reviewers: shankare, ruiu, kparzysz, bcain.
Herald added a subscriber: cfe-commits.
Use GetLinkerPath method instead of hard-coding linker name.
Change should allow -fuse-ld to work on the Hexagon target.
Repository:
rC Clang
https://reviews.llvm.
JonasToth added a comment.
Hi ymandel,
welcome to the LLVM community and thank you very much for working on that check!
If you have any questions or other issues don't hesitate to ask ;)
Comment at: clang-tidy/readability/ConstValueReturnCheck.cpp:25
+
+// Finds the location
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
https://reviews.llvm.org/D52990
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Szelethus added a comment.
Thanks!
I admit that the difficulty was mostly chosen at random, so that could be
brought closer to the actual difficulty of the project.
Comment at: www/analyzer/open_projects.html:86-87
+the opposite direction - integers to pointers - are comp
thiagomacieira created this revision.
thiagomacieira added reviewers: erichkeane, craig.topper.
Herald added a subscriber: cfe-commits.
craig.topper retitled this revision from "Remove FeatureRTM from Skylake
processor list" to "[X86] Remove FeatureRTM from Skylake processor list".
There are a LO
craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rC Clang
https://reviews.llvm.org/D53042
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llv
lebedev.ri added inline comments.
Comment at: lib/Basic/Targets/X86.cpp:169
setFeatureEnabledImpl(Features, "mpx", true);
if (Kind != CK_SkylakeServer) // SKX inherits all SKL features, except SGX
setFeatureEnabledImpl(Features, "sgx", true);
> h
george.karpenkov added a comment.
> there's a way to run clang-tidy over all of clang+llvm automatically
cmake should generate compile_commands.json by default, and then you could just
point clang-tidy at that.
Repository:
rC Clang
https://reviews.llvm.org/D52905
steveire added a comment.
> What's more, given that clang-output has no real documentation to speak of,
> how will users even *know* to update their scripts?
The release notes will tell them.
But your comment also raises a different point: If there is so little
clang-query documentation, what
erik.pilkington created this revision.
erik.pilkington added reviewers: rsmith, rjmccall.
Herald added a subscriber: dexonsmith.
Clang used to error out on the attached testcase, due to multiple definitions
of `foo`. The problem is that multiple FunctionTemplateDecl::Common
pointers are created
erik.pilkington added inline comments.
Comment at: clang/lib/Sema/SemaDecl.cpp:10015
// merged.
if (MergeFunctionDecl(NewFD, OldDecl, S, MergeTypeWithPrevious)) {
NewFD->setInvalidDecl();
The problem is here, MergeFunctionDecl() needs the injecte
erik.pilkington created this revision.
erik.pilkington added reviewers: rsmith, aaron.ballman.
Herald added a subscriber: dexonsmith.
These two diagnostics are noisy, so its reasonable for users to opt-out of them
when -Wconversion is enabled.
Fixes rdar://45058981
Thanks for taking a look!
R
Szelethus updated this revision to Diff 168885.
Szelethus added a comment.
Added new guards as recommended by @xazax.hun
https://reviews.llvm.org/D51866
Files:
lib/StaticAnalyzer/Checkers/UninitializedObject/UninitializedObject.h
lib/StaticAnalyzer/Checkers/UninitializedObject/Uninitialized
Szelethus added inline comments.
Comment at:
lib/StaticAnalyzer/Checkers/UninitializedObject/UninitializedObjectChecker.cpp:519
+
+if (FirstAccess->getBeginLoc() < FirstGuard->getBeginLoc())
+ return true;
george.karpenkov wrote:
> Szelethus wrote:
> >
lebedev.ri added a comment.
In https://reviews.llvm.org/D52998#1258962, @eandrews wrote:
> Yes. I understand. At the moment, exception handling flags are generated
> based on `BENCHMARK_ENABLE_EXCEPTIONS` in `utils/benchmark/CMakeLists.txt` .
> However, `_HAS_EXCEPTIONS` is not defined based
lebedev.ri added a comment.
LG other than the `hasCanonicalType()` vs `hasType()` question.
https://reviews.llvm.org/D52727
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MaskRay updated this revision to Diff 168898.
MaskRay retitled this revision from "[cc1][cc1as] Call OptTable::PrintHelp with
explicit " [options] "" to "[cc1][cc1as] Call OptTable::PrintHelp with
explicit " [options] file..."".
MaskRay removed a reviewer: clang.
MaskRay removed subscribers: sdar
dexonsmith added inline comments.
Comment at: clang/test/Sema/implicit-int-conversion.c:1-4
+// RUN: %clang_cc1 -Wconversion -Wno-implicit-int-conversion -DSMALL=char
-DBIG=int -DNO_DIAG
+// RUN: %clang_cc1 -Wno-conversion -Wimplicit-int-conversion -DSMALL=char
-DBIG=int
+// RU
erik.pilkington added inline comments.
Comment at: clang/test/Sema/implicit-int-conversion.c:1-4
+// RUN: %clang_cc1 -Wconversion -Wno-implicit-int-conversion -DSMALL=char
-DBIG=int -DNO_DIAG
+// RUN: %clang_cc1 -Wno-conversion -Wimplicit-int-conversion -DSMALL=char
-DBIG=int
+
erik.pilkington updated this revision to Diff 168912.
erik.pilkington added a comment.
Actually run the tests!
https://reviews.llvm.org/D53048
Files:
clang/include/clang/Basic/DiagnosticGroups.td
clang/include/clang/Basic/DiagnosticSemaKinds.td
clang/test/Sema/implicit-int-conversion.c
MaskRay updated this revision to Diff 168913.
MaskRay retitled this revision from "[cc1][cc1as] Call OptTable::PrintHelp with
explicit " [options] file..."" to "[Driver][cc1][cc1as] Call
OptTable::PrintHelp with explicit " [options] file..."".
MaskRay added a comment.
Another reference in driver
leonardchan added a comment.
@ebevhan @rjmccall Seeing that the saturation intrinsic in
https://reviews.llvm.org/D52286 should be put on hold for now, would it be fine
to submit this patch as is? Then if the intrinsic is finished, come back to
this and update this patch to use the intrinsic?
Author: maskray
Date: Tue Oct 9 17:15:33 2018
New Revision: 344098
URL: http://llvm.org/viewvc/llvm-project?rev=344098&view=rev
Log:
[Driver][cc1][cc1as] Call OptTable::PrintHelp with explicit " [options] file..."
Summary: This is to accommodate a change in llvm/lib/Option/OptTable.cpp D51009
R
This revision was automatically updated to reflect the committed changes.
Closed by commit rL344098: [Driver][cc1][cc1as] Call OptTable::PrintHelp with
explicit " [options] file..." (authored by MaskRay, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://revie
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
I wonder if we should have a `-fno-strict-something` flag for this. I suppose
there's no more effective way to find out than to try this and see what happens.
Comment at: li
krytarowski added inline comments.
Comment at: lib/CodeGen/BackendUtil.cpp:323
Opts.ToolType = EfficiencySanitizerOptions::ESAN_CacheFrag;
- else if (LangOpts.Sanitize.has(SanitizerKind::EfficiencyWorkingSet))
+ else if (T.getOS() == Triple::Linux &&
+LangOpts.Sanitize
xbolva00 added a comment.
ping
https://reviews.llvm.org/D52949
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: emaste
Date: Tue Oct 9 17:34:17 2018
New Revision: 344100
URL: http://llvm.org/viewvc/llvm-project?rev=344100&view=rev
Log:
clang: Allow ifunc resolvers to accept arguments
When ifunc support was added to Clang (r265917) it did not allow
resolvers to take function arguments. This was ba
This revision was automatically updated to reflect the committed changes.
Closed by commit rC344100: clang: Allow ifunc resolvers to accept arguments
(authored by emaste, committed by ).
Repository:
rC Clang
https://reviews.llvm.org/D52703
Files:
include/clang/Basic/AttrDocs.td
include/cl
1 - 100 of 155 matches
Mail list logo
