mstorsjo added inline comments.
Comment at: tools/clang/lib/Driver/ToolChain.cpp:394
+llvm::SmallString<8> LinkerName;
+if(Triple.isOSDarwin())
+ LinkerName.append("ld64.");
Space between `if` and `(`
Repository:
rL LLVM
https://reviews.llvm.org
krasimir accepted this revision.
krasimir added a comment.
This revision is now accepted and ready to land.
Looks good!
I must have gotten confused with my test suggestion.
Do you have commit access, or should I commit this for you?
https://reviews.llvm.org/D37904
___
Author: erichkeane
Date: Tue Sep 26 20:23:02 2017
New Revision: 314268
URL: http://llvm.org/viewvc/llvm-project?rev=314268&view=rev
Log:
Add test forgotten in r314262.
Added:
cfe/trunk/test/CodeGenCXX/extern-section-attribute.cpp (with props)
Added: cfe/trunk/test/CodeGenCXX/extern-section
Author: erichkeane
Date: Tue Sep 26 20:20:13 2017
New Revision: 314267
URL: http://llvm.org/viewvc/llvm-project?rev=314267&view=rev
Log:
Fix capitalization of a bunch of parameters in SemaDeclAttr [NFC]
Modified:
cfe/trunk/lib/Sema/SemaDeclAttr.cpp
Modified: cfe/trunk/lib/Sema/SemaDeclAttr.c
gtbercea updated this revision to Diff 116747.
gtbercea added a comment.
Fix test.
https://reviews.llvm.org/D38040
Files:
test/OpenMP/openmp_offload_codegen.cpp
Index: test/OpenMP/openmp_offload_codegen.cpp
===
--- /dev/null
++
Author: chh
Date: Tue Sep 26 17:58:45 2017
New Revision: 314264
URL: http://llvm.org/viewvc/llvm-project?rev=314264&view=rev
Log:
[clang-format] Adjust space around &/&& of structured bindings
Keep space before or after the &/&& tokens, but not both. For example,
auto [x,y] = a;
auto &[xr, yr
This revision was automatically updated to reflect the committed changes.
Closed by commit rL314264: [clang-format] Adjust space around &/&& of
structured bindings (authored by chh).
Changed prior to commit:
https://reviews.llvm.org/D35743?vs=116201&id=116744#toc
Repository:
rL LLVM
https:/
gtbercea reopened this revision.
gtbercea added a comment.
This revision is now accepted and ready to land.
Open
https://reviews.llvm.org/D38040
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/c
I'm currently at cppcon with limited bandwidth for reviews. I believe that
the literal test that I mentioned put inside the namespace fixer test suite
fails before your changes, could you please double check?
On Tue, 26 Sep 2017, 15:30 Marek Kurdej via Phabricator, <
revi...@reviews.llvm.org> wrot
krasimir updated this revision to Diff 116742.
krasimir added a comment.
- Change bool from Paren to Line and generalize it
https://reviews.llvm.org/D37695
Files:
lib/Format/BreakableToken.cpp
lib/Format/BreakableToken.h
lib/Format/ContinuationIndenter.cpp
lib/Format/ContinuationIndente
aprantl added a comment.
Does this have to be exposed through the driver or could this be a cc1 option
only?
Comment at: include/clang/Basic/LangOptions.def:144
BENIGN_LANGOPT(EmitAllDecls , 1, 0, "emitting all declarations")
+BENIGN_LANGOPT(EmitFwdTemplateChildren, 1, 0
dcoughlin accepted this revision.
dcoughlin added a comment.
LGTM! Thanks Artem.
https://reviews.llvm.org/D37023
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
BTW: The guy who told me about this issue ALSO discovered a different issue
that that I didn't solve due to his reproducer not contining the whole
testcase. I've got a patch to fix all of it in his hands right now, so I'll
submit this fix with it together.
-Erich
-Original Message-
Fr
This revision was automatically updated to reflect the committed changes.
Closed by commit rL314262: Emit section information for extern variables.
(authored by erichkeane).
Changed prior to commit:
https://reviews.llvm.org/D36487?vs=116581&id=116739#toc
Repository:
rL LLVM
https://reviews
Author: erichkeane
Date: Tue Sep 26 16:42:34 2017
New Revision: 314262
URL: http://llvm.org/viewvc/llvm-project?rev=314262&view=rev
Log:
Emit section information for extern variables.
Currently, if _attribute_((section())) is used for extern variables,
section information is not emitted in gene
wmi added a comment.
In https://reviews.llvm.org/D36562#880808, @hfinkel wrote:
> You seem to be only changing the behavior for the "separatable" fields, but I
> suspect you want to change the behavior for the others too. The bitfield
> would be decomposed into shards, separated by the naturall
Author: jvesely
Date: Tue Sep 26 16:15:54 2017
New Revision: 314261
URL: http://llvm.org/viewvc/llvm-project?rev=314261&view=rev
Log:
Add missing HAVE_LLVM define to fix build with latest llvm
Broken since r314111
V2: pointed out by Jan Vesely
- Use format() instead of % formating
Patch-by:
Author: stl_msft
Date: Tue Sep 26 16:08:43 2017
New Revision: 314259
URL: http://llvm.org/viewvc/llvm-project?rev=314259&view=rev
Log:
[libcxx] [test] Fix unused local typedef warnings.
Modified:
libcxx/trunk/test/std/strings/basic.string/string.ops/string_find.last.not.of/string_view_size.p
Author: stl_msft
Date: Tue Sep 26 16:08:41 2017
New Revision: 314258
URL: http://llvm.org/viewvc/llvm-project?rev=314258&view=rev
Log:
[libcxx] [test] Strip trailing whitespace.
Modified:
libcxx/trunk/test/std/strings/basic.string/string.modifiers/string_insert/string_view.pass.cpp
Modified
Author: stl_msft
Date: Tue Sep 26 16:08:39 2017
New Revision: 314257
URL: http://llvm.org/viewvc/llvm-project?rev=314257&view=rev
Log:
[libcxx] [test] Silence warning C4324 for MSVC.
This warning "structure was padded due to alignment specifier" says
that the compiler is going to do exactly what
efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D36487
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/c
This revision was automatically updated to reflect the committed changes.
Closed by commit rL314229: [Sema] Corrected the warn-on-throw-from-noexcept
behavior to include nothrow (authored by erichkeane).
Changed prior to commit:
https://reviews.llvm.org/D38203?vs=116426&id=116729#toc
Repositor
This revision was automatically updated to reflect the committed changes.
Closed by commit rL314211: [clang-format] Add ext/ to google include categories
(authored by krasimir).
Repository:
rL LLVM
https://reviews.llvm.org/D38243
Files:
cfe/trunk/lib/Format/Format.cpp
Index: cfe/trunk/lib
This revision was automatically updated to reflect the committed changes.
Closed by commit rL314218: [analyzer] Keep track of design discusions as part
of analyzer documentation. (authored by dergachev).
Changed prior to commit:
https://reviews.llvm.org/D36737?vs=116520&id=116724#toc
Repositor
This revision was automatically updated to reflect the committed changes.
Closed by commit rL314213: [mips] Accept but ignore -m(no-)branch-likely
(authored by sdardis).
Repository:
rL LLVM
https://reviews.llvm.org/D38168
Files:
cfe/trunk/include/clang/Driver/Options.td
cfe/trunk/test/Dri
This revision was automatically updated to reflect the committed changes.
Closed by commit rL314209: Fix TBAA information for reference accesses
(authored by kosarev).
Changed prior to commit:
https://reviews.llvm.org/D38074?vs=116326&id=116721#toc
Repository:
rL LLVM
https://reviews.llvm.o
MatzeB added inline comments.
Comment at: lib/CodeGen/CodeGenModule.cpp:477
Context.getTypeSizeInChars(Context.getWideCharType()).getQuantity();
- assert((LangOpts.ShortWChar ||
- llvm::TargetLibraryInfoImpl::getTargetWCharSize(Target.getTriple())
==
---
probinson updated this revision to Diff 116711.
probinson added a comment.
Add a command-line flag to control emitting the template parameter children.
Default to on for SCE debugger tuning.
I am not super excited about my choice of option name or the help text;
alternate suggestions welcome.
I
erichkeane added a comment.
In https://reviews.llvm.org/D38209#880559, @aaron.ballman wrote:
> In https://reviews.llvm.org/D38209#880553, @STL_MSFT wrote:
>
> > > do you think `__declspec(nothrow)` calling the terminate handler in Clang
> > > is a bug?
> >
> > It's certainly a behavior differenc
MatzeB closed this revision.
MatzeB added a comment.
This was accepted on the mailinglist and committed in r294065
Repository:
rL LLVM
https://reviews.llvm.org/D29479
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/c
yaxunl marked an inline comment as done.
yaxunl added inline comments.
Comment at: lib/Sema/SemaType.cpp:6974
+ if (state.getSema().getLangOpts().OpenCL && !hasOpenCLAddressSpace &&
+ type.getAddressSpace() == LangAS::Default &&
Anastasia wrote:
> I am no
erichkeane updated this revision to Diff 116705.
erichkeane added a comment.
Added declspec, fixed does cannot.
https://reviews.llvm.org/D38202
Files:
include/clang/Basic/Attr.td
include/clang/Basic/AttrDocs.td
test/Misc/pragma-attribute-supported-attributes-list.test
Index: test/Misc/p
yaxunl updated this revision to Diff 116704.
yaxunl marked 5 inline comments as done.
yaxunl added a comment.
Rebase to ToT and clean up logic.
https://reviews.llvm.org/D35082
Files:
include/clang/AST/ASTContext.h
include/clang/AST/Type.h
include/clang/Basic/AddressSpaces.h
lib/AST/ASTC
erichkeane marked an inline comment as done.
erichkeane added inline comments.
Comment at: include/clang/Basic/AttrDocs.td:2692
+ let Content = [{
+Clang supports the GNU style ``__attribute__((nothrow))`` attribute as an
+equivilent of `noexcept` on function declarations. This
thakis created this revision.
Herald added a subscriber: klimek.
https://reviews.llvm.org/rL299952 merged '>>>' tokens into a single
JavaRightLogicalShift token. This broke formatting of generics nested more than
two deep, e.g. `Foo>>` because the '>>>' now weren't three '>' for
parseAngle().
martell added a comment.
I have a patch that does a bunch of updates for testing lld correctly in clang
and this was a part of it but I didn't want the patch to get too involved so I
separated this out.
I have seen this issue raised on various forums such as stackoverflow here and
figured I wou
aaron.ballman added a comment.
In https://reviews.llvm.org/D37436#871117, @aaron.ballman wrote:
> Updated based on review feedback.
>
> - Rename CAttributes to DoubleSquareBracketAttributes
> - Added Objective-C test cases to demonstrate no regressed behavior (based on
> a use-case pointed out d
gtbercea reopened this revision.
gtbercea added a comment.
This revision is now accepted and ready to land.
Open
Repository:
rL LLVM
https://reviews.llvm.org/D29660
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi
leanil added a comment.
In https://reviews.llvm.org/D38171#880021, @xazax.hun wrote:
> In https://reviews.llvm.org/D38171#878808, @alexfh wrote:
>
> > András, that's definitely an interesting idea. However, it might be
> > interesting to explore a more principled approach:
> >
> > 1. Make `clang
martell created this revision.
Herald added a subscriber: mgorny.
Currently when we do `-fuse-ld=lld` for apple targets this invokes the gnu
frontend because of the `ld.lld` alias
This creates a new alias for ld64 as `ld64.lld` to avoid this.
Repository:
rL LLVM
https://reviews.llvm.org/D382
leanil updated this revision to Diff 116694.
leanil added a comment.
Add more tests and minor fixes.
https://reviews.llvm.org/D38171
Files:
clang-tidy/ClangTidy.cpp
clang-tidy/ClangTidyDiagnosticConsumer.cpp
clang-tidy/ClangTidyDiagnosticConsumer.h
clang-tidy/ClangTidyModule.h
clang-t
tra closed this revision.
tra added a comment.
Landed with fix in r314223.
https://reviews.llvm.org/D38191
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
spyffe updated this revision to Diff 116680.
spyffe added a comment.
Updates and small refactors requested by Bruno.
Repository:
rL LLVM
https://reviews.llvm.org/D38208
Files:
include/clang/AST/ExternalASTMerger.h
lib/AST/ExternalASTMerger.cpp
lib/Sema/SemaType.cpp
test/Import/extern
NoQ updated this revision to Diff 116673.
NoQ added a comment.
Add no-crash test cases from https://bugs.llvm.org/show_bug.cgi?id=34373 and
https://bugs.llvm.org/show_bug.cgi?id=34731 .
https://reviews.llvm.org/D37023
Files:
lib/StaticAnalyzer/Core/BugReporterVisitors.cpp
test/Analysis/nul
tra updated this revision to Diff 116674.
tra added a comment.
Added missing return. Tests pass now.
https://reviews.llvm.org/D38191
Files:
clang/include/clang/Basic/BuiltinsNVPTX.def
clang/lib/CodeGen/CGBuiltin.cpp
clang/lib/Headers/__clang_cuda_intrinsics.h
clang/test/CodeGen/builtins
gtbercea updated this revision to Diff 116671.
gtbercea added a comment.
Add nocudalib flag.
https://reviews.llvm.org/D38040
Files:
test/OpenMP/target_map_codegen.cpp
Index: test/OpenMP/target_map_codegen.cpp
===
--- test/OpenM
gtbercea reopened this revision.
gtbercea added a comment.
This revision is now accepted and ready to land.
Open
https://reviews.llvm.org/D38040
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/c
Hahnfeld accepted this revision.
Hahnfeld added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D38040
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/c
dberris closed this revision.
dberris added a comment.
Already landed; manually closing.
https://reviews.llvm.org/D38226
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
gtbercea updated this revision to Diff 116664.
gtbercea added a comment.
Fix test.
https://reviews.llvm.org/D38040
Files:
test/OpenMP/target_map_codegen.cpp
Index: test/OpenMP/target_map_codegen.cpp
===
--- test/OpenMP/target_m
malcolm.parsons added inline comments.
Comment at: clang-tidy/modernize/UseEqualsDefaultCheck.cpp:117
+ const CXXMethodDecl *Operator) {
+ const auto *Record = Operator->getParent();
+
aaron.ballman wrote:
> Please don
djasper added inline comments.
Comment at: lib/Format/ContinuationIndenter.h:270
+ // \c breakProtrudingToken.
+ bool LastBlockCommentWasBroken : 1;
+
We should be *very* careful when adding stuff to ParenState as every extra bit
of data and every extra compar
baloghadamsoftware updated this revision to Diff 116662.
baloghadamsoftware added a comment.
If both sides have concrete integers, they must be in range [min/4..max/4], if
only one, it must be in range [min/2..max/2].
https://reviews.llvm.org/D35109
Files:
lib/StaticAnalyzer/Core/SimpleSValB
djasper accepted this revision.
djasper added a comment.
This revision is now accepted and ready to land.
Looks good.
https://reviews.llvm.org/D38243
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listi
Hahnfeld created this revision.
Herald added subscribers: mgorny, dberris, nemanjai.
TEST_BIG_ENDIAN() performs compile tests that will fail with
-nodefaultlibs when building under LLVM_USE_SANITIZER.
https://reviews.llvm.org/D38277
Files:
cmake/base-config-ix.cmake
Index: cmake/base-config
curdeius accepted this revision.
curdeius added a comment.
This revision is now accepted and ready to land.
Thanks for the changes. LGTM.
https://reviews.llvm.org/D35743
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/c
mstorsjo added a comment.
In https://reviews.llvm.org/D38250#880871, @compnerd wrote:
> I have a complete implementation for this which is known to work on Windows
> at least.
Patches for libunwind, or standalone? In any case, please do share if you think
it's relevant.
> The only thing that
curdeius added a comment.
Any thoughts on this?
https://reviews.llvm.org/D37904
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mstorsjo closed this revision.
mstorsjo added a comment.
Committed in SVN r314196.
https://reviews.llvm.org/D38247
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mstorsjo closed this revision.
mstorsjo marked an inline comment as done.
mstorsjo added a comment.
Committed in SVN r314197.
https://reviews.llvm.org/D38249
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailm
ioeric accepted this revision.
ioeric added a comment.
This revision is now accepted and ready to land.
@klimek Have you got a chance to take a look?
Repository:
rL LLVM
https://reviews.llvm.org/D37681
___
cfe-commits mailing list
cfe-commits@lis
ioeric accepted this revision.
ioeric added a comment.
This revision is now accepted and ready to land.
Lgtm.
Thanks for the documentation! Now we just need to try to keep it up-to-date ;)
Comment at: docs/RefactoringEngine.rst:33
+operations (rules). These rules are grouped u
compnerd requested changes to this revision.
compnerd added a comment.
This revision now requires changes to proceed.
I have a complete implementation for this which is known to work on Windows at
least. The only thing that `__thread` requires is a working linker (which does
not include binutil
compnerd accepted this revision.
compnerd added inline comments.
This revision is now accepted and ready to land.
Comment at: src/libunwind.cpp:27
+#ifndef __USING_SJLJ_EXCEPTIONS__
#include "AddressSpace.hpp"
I would prefer that you used:
#if !defined(__
tra reopened this revision.
tra added inline comments.
This revision is now accepted and ready to land.
Comment at: llvm/trunk/lib/Target/NVPTX/NVPTXISelDAGToDAG.cpp:716
case Intrinsic::nvvm_texsurf_handle_internal:
SelectTexSurfHandle(N);
+ case Intrinsic::nvvm_match_al
alexshap updated this revision to Diff 116635.
alexshap added a comment.
fix typo
Repository:
rL LLVM
https://reviews.llvm.org/D38270
Files:
include/clang/AST/ASTContext.h
include/clang/Basic/TargetInfo.h
lib/AST/ASTContext.cpp
Index: lib/AST/ASTContext.cpp
==
dberris updated this revision to Diff 116637.
dberris added a comment.
Herald added a subscriber: srhines.
- fixup: use supported target list in test from clang implementation
https://reviews.llvm.org/D38226
Files:
lib/Driver/ToolChains/Gnu.cpp
test/Driver/XRay/lit.local.cfg
test/Driver/X
alexshap created this revision.
C11 standard refers to the unsigned counterpart of the type ptrdiff_t
in the paragraph 7.19.6 where it defines the format specifier %tu.
In Clang (in PrintfFormatString.cpp, lines 508-510) there is a FIXME for this
case,
in particular, right now Clang doesn't diag
hfinkel added a comment.
You seem to be only changing the behavior for the "separatable" fields, but I
suspect you want to change the behavior for the others too. The bitfield would
be decomposed into shards, separated by the naturally-sized-and-aligned fields.
Each access only loads its shard.
dberris updated this revision to Diff 116631.
dberris added a comment.
- fixup: limit tests to run on supported platforms and oses
https://reviews.llvm.org/D38226
Files:
lib/Driver/ToolChains/Gnu.cpp
test/Driver/XRay/lit.local.cfg
test/Driver/XRay/xray-shared-noxray.cpp
Index: test/Driv
Author: rsmith
Date: Tue Sep 26 14:33:43 2017
New Revision: 314246
URL: http://llvm.org/viewvc/llvm-project?rev=314246&view=rev
Log:
Fix uninitialized member found by msan build bot.
Modified:
cfe/trunk/lib/Sema/SemaOverload.cpp
Modified: cfe/trunk/lib/Sema/SemaOverload.cpp
URL:
http://llvm
Author: compnerd
Date: Tue Sep 26 13:12:04 2017
New Revision: 314241
URL: http://llvm.org/viewvc/llvm-project?rev=314241&view=rev
Log:
Sema: rename SemaBuiltinVAStart to SemaBuiltinVAStartMicrosoft
This function is used to perform semantic analysis on Microsoft style
`__va_start`. Rename it to m
Ah! Sorry, I didn't realize that isExternCXXContext actually searched the
entire search-set! Followup patch coming Thanks for the heads up!
-Original Message-
From: Friedman, Eli [mailto:efrie...@codeaurora.org]
Sent: Tuesday, September 26, 2017 12:06 PM
To: Keane, Erich ; cfe-commits@
Author: compnerd
Date: Tue Sep 26 12:26:01 2017
New Revision: 314236
URL: http://llvm.org/viewvc/llvm-project?rev=314236&view=rev
Log:
Basic: support Preserve{Most,All} CC on Windows
Add support for the `preserve_mostcc` and `preserve_allcc` on Windows
x86_64 and AArch64. This is used by Swift.
On 9/26/2017 11:55 AM, Erich Keane via cfe-commits wrote:
Author: erichkeane
Date: Tue Sep 26 11:55:16 2017
New Revision: 314235
URL: http://llvm.org/viewvc/llvm-project?rev=314235&view=rev
Log:
Allow IUnknown/IInterface types to come from extern C++
It was brought up in response to my last imp
Author: erichkeane
Date: Tue Sep 26 11:55:16 2017
New Revision: 314235
URL: http://llvm.org/viewvc/llvm-project?rev=314235&view=rev
Log:
Allow IUnknown/IInterface types to come from extern C++
It was brought up in response to my last implementation for
this struct-as-interface features that at l
On Tue, Sep 26, 2017 at 11:10 AM, David Majnemer
wrote:
> Shouldn't you only loosen the check for things targeting the Windows SDK?
> GNU platforms shouldn't need this.
>
That code path only actually happens for MS mode. `__va_start` is marked
as `ALL_MS_LANGUAGES`. I suppoes that it would be
Author: nico
Date: Tue Sep 26 11:38:56 2017
New Revision: 314232
URL: http://llvm.org/viewvc/llvm-project?rev=314232&view=rev
Log:
Delete trailing whitespace.
Modified:
cfe/trunk/lib/Lex/PPMacroExpansion.cpp
Modified: cfe/trunk/lib/Lex/PPMacroExpansion.cpp
URL:
http://llvm.org/viewvc/llvm-p
Author: rsmith
Date: Tue Sep 26 11:37:55 2017
New Revision: 314231
URL: http://llvm.org/viewvc/llvm-project?rev=314231&view=rev
Log:
Resolve a defect in C++17 copy omission.
When selecting constructors for initializing an object of type T from a single
expression of class type U, also consider co
Author: erichkeane
Date: Tue Sep 26 11:20:39 2017
New Revision: 314229
URL: http://llvm.org/viewvc/llvm-project?rev=314229&view=rev
Log:
[Sema] Corrected the warn-on-throw-from-noexcept behavior to include nothrow
Discovered that 'nothrow' (which is supposed to be an alias for noexcept)
was not w
Shouldn't you only loosen the check for things targeting the Windows SDK?
GNU platforms shouldn't need this.
On Tue, Sep 26, 2017 at 10:44 AM, Saleem Abdulrasool via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: compnerd
> Date: Tue Sep 26 10:44:10 2017
> New Revision: 314226
>
> URL
Author: compnerd
Date: Tue Sep 26 10:44:10 2017
New Revision: 314226
URL: http://llvm.org/viewvc/llvm-project?rev=314226&view=rev
Log:
Sema: Windows/ARM __va_start is not const correct
The `__va_start` intrinsic for Windows ARM does not account for const
correctness when performing a check. All
Author: tra
Date: Tue Sep 26 10:07:23 2017
New Revision: 314223
URL: http://llvm.org/viewvc/llvm-project?rev=314223&view=rev
Log:
[NVPTX] added match.{any,all}.sync instructions, intrinsics & builtins.
Differential Revision: https://reviews.llvm.org/D38191
Modified:
cfe/trunk/include/clang/B
Author: abataev
Date: Tue Sep 26 09:19:04 2017
New Revision: 314220
URL: http://llvm.org/viewvc/llvm-project?rev=314220&view=rev
Log:
[OPENMP] Fix handling of implicit mapping of array sections.
Modified:
cfe/trunk/lib/Sema/SemaOpenMP.cpp
cfe/trunk/test/OpenMP/target_map_messages.cpp
Mod
Another bit of feedback: The Microsoft SDK has pragma adjustment headers
that are used like so:
#include
// Define structs with alignment 1 here
#include
This warning fires on that. I haven't seen these headers in active use
recently, but I remember seeing them quite a bit maybe 10 years ago. (
Author: dergachev
Date: Tue Sep 26 08:49:53 2017
New Revision: 314218
URL: http://llvm.org/viewvc/llvm-project?rev=314218&view=rev
Log:
[analyzer] Keep track of design discusions as part of analyzer documentation.
Create a directory to store discussions on potentially useful features that are
not
Author: sdardis
Date: Tue Sep 26 08:01:21 2017
New Revision: 314213
URL: http://llvm.org/viewvc/llvm-project?rev=314213&view=rev
Log:
[mips] Accept but ignore -m(no-)branch-likely
-mbranch-likely and -mno-branch-likely are used in some build systems for
some MIPS targets. Accept these options but
Author: krasimir
Date: Tue Sep 26 07:58:29 2017
New Revision: 314211
URL: http://llvm.org/viewvc/llvm-project?rev=314211&view=rev
Log:
[clang-format] Add ext/ to google include categories
Summary: This adds an ext/ header include category for google style.
Reviewers: djasper
Reviewed By: djaspe
Author: kosarev
Date: Tue Sep 26 07:22:48 2017
New Revision: 314209
URL: http://llvm.org/viewvc/llvm-project?rev=314209&view=rev
Log:
Fix TBAA information for reference accesses
This patch fixes clang to decorate reference accesses as pointers
and not as "omnipotent chars".
Differential Revision
Author: abataev
Date: Tue Sep 26 06:47:31 2017
New Revision: 314205
URL: http://llvm.org/viewvc/llvm-project?rev=314205&view=rev
Log:
[OPENMP] Generate implicit map|firstprivate clauses for target-based
directives.
If the variable is used in the target-based region but is not found in
any private
Author: mstorsjo
Date: Tue Sep 26 01:07:26 2017
New Revision: 314197
URL: http://llvm.org/viewvc/llvm-project?rev=314197&view=rev
Log:
Skip building unused parts when targeting SJLJ
When SJLJ exceptions are used, those functions aren't used.
This fixes build failures on ARM with SJLJ enabled (e.
Author: mstorsjo
Date: Tue Sep 26 01:07:17 2017
New Revision: 314196
URL: http://llvm.org/viewvc/llvm-project?rev=314196&view=rev
Log:
Correct data types in the _Unwind_FunctionContext struct
This makes it match the definition used within llvm and in libgcc,
we previously got the wrong layout in
92 matches
Mail list logo
