djasper accepted this revision.
djasper added a comment.
This revision is now accepted and ready to land.
Looks good.
https://reviews.llvm.org/D19385
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listi
djasper added a comment.
Ping?
https://reviews.llvm.org/D21026
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
djasper added a comment.
I think we should add a test for why this was a problem. IIUC, this is caused
by something like:
#if a
#else
#endif
namespace {
}
Because we actually analyze the whole thing twice.
https://reviews.llvm.org/D25162
__
djasper added a comment.
Ping?
SpaceAfterTemplateKeyword exists by now.
https://reviews.llvm.org/D19066
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
djasper accepted this revision.
djasper added a comment.
This revision is now accepted and ready to land.
Sorry for the delay. Looks good.
https://reviews.llvm.org/D19063
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/
plevine created this revision.
plevine added a reviewer: chandlerc.
plevine added a subscriber: cfe-commits.
Herald added subscribers: mgorny, beanz.
LLVM sets CMAKE_INSTALL_RPATH in its top-level CMakeLists.txt to
"\$ORIGIN/../lib${LLVM_LIBDIR_SUFFIX}". When Clang is built with LLVM, this
prop
djasper added a comment.
Could you read:
http://clang.llvm.org/docs/ClangFormatStyleOptions.html#adding-additional-style-options
And provide some evidence about the requirements for new style options?
https://reviews.llvm.org/D25171
___
cfe-commits
djasper added a comment.
So sorry. Seems I forgot to hit "Submit" :(.
If you don't like the ".first" and ".second" of the pair, you could introduce a
struct for it and overload operator<. Might actually be more readable.
> WhitespaceManager.cpp:73
> + Tok.NestingLevel,
>/*Spaces=
bmharper added a comment.
ping!
https://reviews.llvm.org/D21279
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ahatanak added inline comments.
> rsmith wrote in SemaTemplateInstantiateDecl.cpp:4849
> Do we need to do this for conversion function names too? (Eg, `operator
> C1*`)
I added a definition of "operator C1*" and called it in C1::foo1. I didn't
see any crashes or asserts.
Calls to a destructor
dberris added a comment.
In https://reviews.llvm.org/D24799#561106, @rSerge wrote:
> My mistake was that initially I only enumerated the unsupported targets from
> llvm\include\llvm\ADT\Triple.h . Now I've added also the cases from
> llvm\lib\Support\Triple.cpp .
> `XFAIL` requires a list of a
majnemer added inline comments.
> CGBuiltin.cpp:2640-2647
> +// Many of MSVC builtins are on both x64 and ARM; to avoid repeating code, we
> +// handle them here.
> +namespace MSVC {
> + enum {
> +_BitScanForward,
> +_BitScanReverse
> + };
This should be in an anonymous namespace. Also
agutowski created this revision.
agutowski added reviewers: rnk, hans, thakis, majnemer.
agutowski added a subscriber: cfe-commits.
Herald added a subscriber: aemerson.
_BitScan intrinsics (and some others, for example _Interlocked and _bittest)
are supposed to work on both ARM and x86. This is a
Eugene.Zelenko closed this revision.
Eugene.Zelenko added a comment.
Committed in https://reviews.llvm.org/rL191945.
https://reviews.llvm.org/D1752
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinf
Eugene.Zelenko closed this revision.
Eugene.Zelenko added a comment.
Committed in https://reviews.llvm.org/rL191944.
https://reviews.llvm.org/D1753
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinf
Eugene.Zelenko closed this revision.
Eugene.Zelenko added a comment.
Committed in https://reviews.llvm.org/rL191808.
https://reviews.llvm.org/D1801
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinf
hfinkel abandoned this revision.
hfinkel added a comment.
Abandoned in favor of
https://reviews.llvm.org/D25225/https://reviews.llvm.org/D25262.
https://reviews.llvm.org/D19678
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.ll
Eugene.Zelenko closed this revision.
Eugene.Zelenko added a comment.
Committed in https://reviews.llvm.org/rL192028.
https://reviews.llvm.org/D1816
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinf
Eugene.Zelenko closed this revision.
Eugene.Zelenko added a comment.
Committed in https://reviews.llvm.org/rL192029.
https://reviews.llvm.org/D1817
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinf
Eugene.Zelenko closed this revision.
Eugene.Zelenko added a comment.
Committed in https://reviews.llvm.org/rL192141.
https://reviews.llvm.org/D1818
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinf
Eugene.Zelenko added a comment.
This revision now requires review to proceed.
Looks like patch was not committed, but may be irrelevant these days.
https://reviews.llvm.org/D1837
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.l
Eugene.Zelenko closed this revision.
Eugene.Zelenko added a comment.
Committed in https://reviews.llvm.org/rL192410.
https://reviews.llvm.org/D1854
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinf
Eugene.Zelenko closed this revision.
Eugene.Zelenko added a comment.
Obsoleted by https://reviews.llvm.org/D1885.
https://reviews.llvm.org/D1869
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/c
elsteveogrande added a comment.
Thanks -- will do one more pass
> majnemer wrote in PrintPreprocessedOutput.cpp:409-411
> Why not have sanitizePath return a `std::vector`?
Hmm, good idea. Better than caller-passed vectors, and it be whatever size is
needed (in the called method I'd `CharVect
majnemer added inline comments.
> PrintPreprocessedOutput.cpp:101
>PrintPPOutputPPCallbacks(Preprocessor &pp, raw_ostream &os, bool
> lineMarkers,
> - bool defines, bool UseLineDirectives)
> + bool defines, bool dumpIncludeDirectives,
> +
erichkeane marked 16 inline comments as done.
erichkeane added a comment.
Commenting to save my comments (don't seem to survive a refresh). Still
working on non-function mangling.
> rnk wrote in ItaniumMangle.cpp:1203
> What mangling should happen for operator overloads and all other kinds of
Author: jlebar
Date: Tue Oct 4 19:27:38 2016
New Revision: 283280
URL: http://llvm.org/viewvc/llvm-project?rev=283280&view=rev
Log:
[CUDA] Add missing ':' to noexcept.cu test.
Modified:
cfe/trunk/test/CodeGenCUDA/nothrow.cu
Modified: cfe/trunk/test/CodeGenCUDA/nothrow.cu
URL:
http://llvm.o
mclow.lists accepted this revision.
mclow.lists added a comment.
I will land this, and close the other one, too.
https://reviews.llvm.org/D25053
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/c
Eugene.Zelenko closed this revision.
Eugene.Zelenko added a comment.
Committed in https://reviews.llvm.org/rL196189.
https://reviews.llvm.org/D2302
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinf
This revision was automatically updated to reflect the committed changes.
Closed by commit rL283272: [CUDA] Mark device functions as nounwind. (authored
by jlebar).
Changed prior to commit:
https://reviews.llvm.org/D25166?vs=73579&id=73581#toc
Repository:
rL LLVM
https://reviews.llvm.org/D2
Author: jlebar
Date: Tue Oct 4 18:41:49 2016
New Revision: 283272
URL: http://llvm.org/viewvc/llvm-project?rev=283272&view=rev
Log:
[CUDA] Mark device functions as nounwind.
Summary:
This prevents clang from emitting 'invoke's and catch statements.
Things previously mostly worked thanks to TryT
This revision was automatically updated to reflect the committed changes.
Closed by commit rL283271: [CUDA] Destroy deferred diagnostics before
destroying the ASTContext's… (authored by jlebar).
Changed prior to commit:
https://reviews.llvm.org/D25260?vs=73578&id=73580#toc
Repository:
rL LLV
GorNishanov added inline comments.
> majnemer wrote in CGCoroutine.cpp:68-69
> I think that's just `CGM.getContext().getTargetInfo().getSuitableAlign() /
> CGM.getContext().getTargetInfo().getCharWidth()`
Yay! One less FIXME.
I think I will make it to match __STDCPP_DEFAULT_NEW_ALIGNMENT__:
Author: jlebar
Date: Tue Oct 4 18:41:45 2016
New Revision: 283271
URL: http://llvm.org/viewvc/llvm-project?rev=283271&view=rev
Log:
[CUDA] Destroy deferred diagnostics before destroying the ASTContext's
PartialDiagnostic allocator.
Summary:
This will let us (in a separate patch) allocate deferr
Eugene.Zelenko closed this revision.
Eugene.Zelenko added a comment.
Committed in https://reviews.llvm.org/rL196191.
https://reviews.llvm.org/D2304
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinf
jlebar updated this revision to Diff 73579.
jlebar marked an inline comment as done.
jlebar added a comment.
Update tests.
https://reviews.llvm.org/D25166
Files:
clang/lib/CodeGen/CGCall.cpp
clang/lib/CodeGen/CGException.cpp
clang/test/CodeGenCUDA/convergent.cu
clang/test/CodeGenCUDA/de
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
https://reviews.llvm.org/D25260
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rnk accepted this revision.
rnk added a reviewer: rnk.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
> nothrow.cu:16
> +// DEVICE: define void @_Z3foov() [[DEVICE_ATTR:#[0-9]+]]
> +__device__ void foo() { f(); }
> +
I would check for `call void @_Z1fv` here, and ma
majnemer added inline comments.
> CGCoroutine.cpp:68-69
> +void CodeGenFunction::EmitCoroutineBody(const CoroutineBodyStmt &S) {
> + auto *NullPtr = llvm::ConstantPointerNull::get(Builder.getInt8PtrTy());
> + // FIXME: Instead of 0, pass an equivalent of alignas(maxalign_t).
> + auto *CoroId =
Eugene.Zelenko closed this revision.
Eugene.Zelenko added a comment.
Committed in https://reviews.llvm.org/rL196887.
https://reviews.llvm.org/D2344
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinf
jlebar updated this revision to Diff 73578.
jlebar added a comment.
Update comment.
https://reviews.llvm.org/D25260
Files:
clang/include/clang/AST/ASTContext.h
clang/lib/CodeGen/CodeGenModule.cpp
Index: clang/lib/CodeGen/CodeGenModule.cpp
==
jlebar updated this revision to Diff 73577.
jlebar added a comment.
Move everything into codegen.
https://reviews.llvm.org/D25166
Files:
clang/lib/CodeGen/CGCall.cpp
clang/lib/CodeGen/CGException.cpp
clang/test/CodeGenCUDA/convergent.cu
clang/test/CodeGenCUDA/device-var-init.cu
clang/
rnk added inline comments.
> CodeGenModule.cpp:512-513
>}
> + // Clear the deferred diags so they don't outlive the ASTContext from
> whence
> + // they're allocated.
> + DeferredDiags.clear();
The ASTContext outlives CodeGenModule, but the diagnostic allocator in the
ASTContext does no
Eugene.Zelenko closed this revision.
Eugene.Zelenko added a comment.
Committed in https://reviews.llvm.org/rL199208.
https://reviews.llvm.org/D2526
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinf
Eugene.Zelenko added a comment.
Looks like patch was not committed.
https://reviews.llvm.org/D22944
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Eugene.Zelenko added a comment.
Looks like patch was not committed.
https://reviews.llvm.org/D22945
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jlebar updated this revision to Diff 73576.
jlebar marked an inline comment as done.
jlebar added a comment.
Rebase atop https://reviews.llvm.org/D25260, which obviates the need for this
ugly PD allocation dance.
https://reviews.llvm.org/D25139
Files:
clang/include/clang/Basic/DiagnosticSemaK
jlebar added inline comments.
> rnk wrote in Sema.h:9258
> The ASTContext should outlive IRgen, since the AST is allocated in its arena.
> Is there a separate diagnostic memory pool that I don't know about?
You're right, this is a silly bug. Fixed in a separate patch,
https://reviews.llvm.org
jlebar created this revision.
jlebar added a reviewer: rnk.
jlebar added a subscriber: cfe-commits.
This will let us (in a separate patch) allocate deferred diagnostics in
the ASTContext's PartialDiagnostic arena.
https://reviews.llvm.org/D25260
Files:
clang/include/clang/AST/ASTContext.h
c
Eugene.Zelenko added a comment.
Looks like patch was not committed.
https://reviews.llvm.org/D23493
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Eugene.Zelenko added a comment.
Looks like patch was not committed.
Repository:
rL LLVM
https://reviews.llvm.org/D23602
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Eugene.Zelenko closed this revision.
Eugene.Zelenko added a comment.
Committed in https://reviews.llvm.org/rL280306. Please specify "Differential
revision: " as last line of commit message.
https://reviews.llvm.org/D23734
___
cfe-commits mailing li
GorNishanov created this revision.
GorNishanov added reviewers: rsmith, EricWF.
GorNishanov added a subscriber: cfe-commits.
Herald added a subscriber: mehdi_amini.
1. Sema: Add allocation / deallocation substatements.
2. Sema: Add labels to final-suspend and deallocation substatements.
3. Sema: A
This revision was automatically updated to reflect the committed changes.
Closed by commit rL283264: Separate builtins for x84-64 and i386; implement
__mulh and __umulh (authored by agutowski).
Changed prior to commit:
https://reviews.llvm.org/D24598?vs=73521&id=73570#toc
Repository:
rL LLVM
Author: agutowski
Date: Tue Oct 4 17:29:49 2016
New Revision: 283264
URL: http://llvm.org/viewvc/llvm-project?rev=283264&view=rev
Log:
Separate builtins for x84-64 and i386; implement __mulh and __umulh
Summary: We need x86-64-specific builtins if we want to implement some of the
MS intrinsics
rjmccall added a comment.
LGTM, but I'm not a code owner here.
https://reviews.llvm.org/D24012
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
EricWF added a comment.
Feel free to send them in 10-20 at a time. It'll be much faster to review that
way.
https://reviews.llvm.org/D24562
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c
kkwli0 added a comment.
Should we issue a warning message in this case?
https://reviews.llvm.org/D24615
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
EricWF added a comment.
r283260.
https://reviews.llvm.org/D25251
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: zturner
Date: Tue Oct 4 16:27:09 2016
New Revision: 283261
URL: http://llvm.org/viewvc/llvm-project?rev=283261&view=rev
Log:
Make building the clang-tidy VS extension less spammy.
The package that strong name signs the 3rd party references
spams a ton of output to the log, making the bui
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
https://reviews.llvm.org/D24598
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.
LGTM. Although I would have thought the first half of the conjunction would
have prevented MSVC's PP from seeing the `__has_include`.
I'll get to the rest of the patches tomorrow.
https://re
Author: ericwf
Date: Tue Oct 4 16:25:51 2016
New Revision: 283260
URL: http://llvm.org/viewvc/llvm-project?rev=283260&view=rev
Log:
[libcxx] [test] Guard __has_include usage with a macro
Summary: There's a macro scheme already being used for __has_feature etc. Use
it for __has_include too, whic
STL_MSFT created this revision.
STL_MSFT added reviewers: EricWF, mclow.lists.
STL_MSFT added a subscriber: cfe-commits.
There's a macro scheme already being used for __has_feature etc. Use it for
__has_include too, which makes MSVC happy (it doesn't support __has_include
yet, and unguarded use
parallaxe added a comment.
In https://reviews.llvm.org/D23236#557898, @dcoughlin wrote:
> Upon reflection, I don't think this is the right approach.
>
> Desugaring any AttributedType in the return type seems like a really, really
> big hammer and could be an unexpected surprise for future attrib
rmaprath added a comment.
In https://reviews.llvm.org/D24562#561442, @Eugene.Zelenko wrote:
> Looks like patch was not committed.
Need to replicate this to as many tests as possible (~150). I was hoping to do
all that and upload another diff for a final review, couldn't get to it I'm
afraid.
Author: hans
Date: Tue Oct 4 16:01:04 2016
New Revision: 283258
URL: http://llvm.org/viewvc/llvm-project?rev=283258&view=rev
Log:
clang-cl: Make /Fo apply also when using -flto
Modified:
cfe/trunk/lib/Driver/Driver.cpp
cfe/trunk/test/Driver/cl-outputs.c
Modified: cfe/trunk/lib/Driver/Dr
Author: hans
Date: Tue Oct 4 16:00:57 2016
New Revision: 283255
URL: http://llvm.org/viewvc/llvm-project?rev=283255&view=rev
Log:
clang-cl: Expose the -flto option
We could hook up /GL as an alias for -flto, but that might be
confusing, as clang-cl in that mode would not be drop-in compatible
wi
Author: hans
Date: Tue Oct 4 16:01:02 2016
New Revision: 283257
URL: http://llvm.org/viewvc/llvm-project?rev=283257&view=rev
Log:
clang-cl: Use the .obj file extension also in LTO mode
Modified:
cfe/trunk/lib/Driver/Types.cpp
cfe/trunk/test/Driver/cl-outputs.c
Modified: cfe/trunk/lib/Dr
Author: hans
Date: Tue Oct 4 16:01:00 2016
New Revision: 283256
URL: http://llvm.org/viewvc/llvm-project?rev=283256&view=rev
Log:
clang-cl: expose the -fuse-ld option
Modified:
cfe/trunk/include/clang/Driver/Options.td
cfe/trunk/test/Driver/cl-link.c
Modified: cfe/trunk/include/clang/Dr
Eugene.Zelenko closed this revision.
Eugene.Zelenko added a comment.
Committed in https://reviews.llvm.org/rL281666.
https://reviews.llvm.org/D24488
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listin
Eugene.Zelenko added a comment.
Looks like patch was not committed.
https://reviews.llvm.org/D24562
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Anastasia added a comment.
> Regarding the improvement proposed by us which "flatten" captured variables
> into invoke_function argument list and block_literal pointer wouldn't be
> passed as first argument(to invoke_function) anymore. The reason why it
> doesn't require global memory managem
Author: zaks
Date: Tue Oct 4 15:49:31 2016
New Revision: 283253
URL: http://llvm.org/viewvc/llvm-project?rev=283253&view=rev
Log:
[analyzer] Add PostStmt callback for ArraySubscriptExpr
A patch by Jan Smets!
Differential Revision: https://reviews.llvm.org/D25009
Modified:
cfe/trunk/lib/Sta
Eugene.Zelenko added a comment.
Looks like patch was not committed.
https://reviews.llvm.org/D24703
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Eugene.Zelenko closed this revision.
Eugene.Zelenko added a comment.
Committed in https://reviews.llvm.org/rL281979.
https://reviews.llvm.org/D24712
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listin
This revision was automatically updated to reflect the committed changes.
Closed by commit rL283251: [clang] make reciprocal estimate codegen a function
attribute (authored by spatel).
Changed prior to commit:
https://reviews.llvm.org/D24815?vs=73364&id=73548#toc
Repository:
rL LLVM
https:/
Author: spatel
Date: Tue Oct 4 15:44:05 2016
New Revision: 283251
URL: http://llvm.org/viewvc/llvm-project?rev=283251&view=rev
Log:
[clang] make reciprocal estimate codegen a function attribute
The motivation for the change is that we can't have pseudo-global settings
for codegen living in Targe
Author: vedantk
Date: Tue Oct 4 15:36:04 2016
New Revision: 283249
URL: http://llvm.org/viewvc/llvm-project?rev=283249&view=rev
Log:
[ubsan] Disable bounds-check for flexible array ivars
This eliminates a class of false positives for -fsanitize=array-bounds
on instrumented ObjC projects.
Differ
NoQ added inline comments.
> chapuni wrote in CloneDetection.cpp:113
> You should update \param here. See also r283106.
Thanks!! Will do.
Repository:
rL LLVM
https://reviews.llvm.org/D24916
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
vsk added a comment.
Thanks for the review!
I looked at the IR and confirmed that the ivars are laid out in the order
they're defined, that the indirect ivar offsets make sense, and that the
runtime ivar offsets match up with what we expect. E.g;
@"OBJC_IVAR_$_FlexibleArray1.chars" = global
NoQ added a comment.
In https://reviews.llvm.org/D24916#561369, @vitalybuka wrote:
> Out of curiosity, how was MSVC crash solved?
It wasn't, unfortunately; phabricator closed this revision by looking at the
commit that was already reverted, with a delay (see
http://lists.llvm.org/pipermail/cf
This revision was automatically updated to reflect the committed changes.
Closed by commit rL283249: [ubsan] Disable bounds-check for flexible array
ivars (authored by vedantk).
Changed prior to commit:
https://reviews.llvm.org/D7?vs=73508&id=73547#toc
Repository:
rL LLVM
https://review
vitalybuka added a comment.
Out of curiosity, how was MSVC crash solved?
Repository:
rL LLVM
https://reviews.llvm.org/D24916
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Eugene.Zelenko closed this revision.
Eugene.Zelenko added a comment.
Committed in https://reviews.llvm.org/rL282488.
https://reviews.llvm.org/D24961
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listin
Author: djasper
Date: Tue Oct 4 15:18:25 2016
New Revision: 283246
URL: http://llvm.org/viewvc/llvm-project?rev=283246&view=rev
Log:
clang-format: Fix bad multi-variable for-loop formatting.
Before:
for (int*p, *q; p != q; p = p->next) {
After:
for (int *p, *q; p != q; p = p->next) {
Modif
CaseyCarter added a comment.
I'll push a revision with the whitespace changes reverted soon.
> .gitignore:59
> keep.lst
> +.vscode/
This is an editor byproduct, I suppose, not "MSVC libraries test harness". Let
me know if anyone cares and I'll add a comment.
> any:553
> +__tmp._
CaseyCarter updated this revision to Diff 73538.
CaseyCarter added a comment.
Fix inadvertent whitespace changes.
https://reviews.llvm.org/D25249
Files:
.gitignore
include/any
test/libcxx/utilities/any/any.class/any.assign/value.pass.cpp
test/libcxx/utilities/any/any.class/any.cons/valu
vleschuk updated this revision to Diff 73537.
vleschuk added a comment.
Pass alignment in bits instead of bytes to backend (conversion is done when
emitting DW_AT_alignment).
https://reviews.llvm.org/D24426
Files:
lib/CodeGen/CGDebugInfo.cpp
lib/CodeGen/CGDebugInfo.h
test/CodeGen/debug-i
Author: aizatsky
Date: Tue Oct 4 14:19:16 2016
New Revision: 283242
URL: http://llvm.org/viewvc/llvm-project?rev=283242&view=rev
Log:
[sancov] documentation update after r283241
Modified:
cfe/trunk/docs/SanitizerCoverage.rst
Modified: cfe/trunk/docs/SanitizerCoverage.rst
URL:
http://llvm.o
CaseyCarter created this revision.
CaseyCarter added reviewers: EricWF, mclow.lists.
CaseyCarter added a subscriber: cfe-commits.
Changes to non-portable behavior in tests. Inline detailed description of
each change is forthcoming. Changes are almost entirely in test code, with one
behavioral c
STL_MSFT created this revision.
STL_MSFT added reviewers: EricWF, mclow.lists.
STL_MSFT added a subscriber: cfe-commits.
Fix MSVC shadow warnings in a newly added test, by renaming local variables.
https://reviews.llvm.org/D25248
Files:
test/std/utilities/utility/forward/move.pass.cpp
Index
zaks.anna added a comment.
Should this revision be closed?
https://reviews.llvm.org/D23853
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
agutowski added inline comments.
> agutowski wrote in TargetBuiltins.h:100
> Nice, thanks!
> As far as I see, it creates some inconsistency in usage of the word "last",
> because it's used wrong everywhere else - LastTSBuiltin is the number of the
> last target-specific builtin **plus one**, wh
agutowski added inline comments.
> rnk wrote in TargetBuiltins.h:100
> I think this would be better with just one enum to reduce compilation time:
>
> /// \brief X86 builtins
> namespace X86 {
> enum {
> LastTIBuiltin = clang::Builtin::FirstTSBuiltin - 1,
> #define BUILTIN(ID
davidsh updated this revision to Diff 73520.
davidsh added a comment.
Adding a parameter to IgnoreContainers instead of copying the logic.
https://reviews.llvm.org/D24615
Files:
include/clang/AST/Stmt.h
lib/AST/Stmt.cpp
lib/Sema/SemaOpenMP.cpp
test/OpenMP/for_loop_messages.cpp
Index:
Hi Richard,
Thanks for your analysis.
This patch is intended to mitigate use-after-free bugs. In that context a
“define strict behavior for invalid pointer values” we could deploy in
production code would be very useful. Maybe calling this a sanitizer is
misleading, and instead it could be pr
agutowski updated this revision to Diff 73521.
agutowski added a comment.
merge enums
https://reviews.llvm.org/D24598
Files:
include/clang/Basic/BuiltinsX86_64.def
include/clang/Basic/TargetBuiltins.h
lib/Basic/Targets.cpp
lib/CodeGen/CGBuiltin.cpp
lib/Headers/intrin.h
test/CodeGen/
elsteveogrande updated this revision to Diff 73518.
elsteveogrande added a comment.
- Fix a few more style nits according to LLVM style guide.
- Further fixed w/ `clang-format -style=LLVM`, kept (most of) recommended
changes.
- Added more unit tests (more include cases: `#include_next`, `-imacro`
Author: rnk
Date: Tue Oct 4 13:10:23 2016
New Revision: 283235
URL: http://llvm.org/viewvc/llvm-project?rev=283235&view=rev
Log:
Test what happens when tag lookup and redeclaration lookup disagree
Clang has a diagnostic for the what happens when an elaborated type
implicitly creates a tag declar
1 - 100 of 223 matches
Mail list logo
