Quuxplusone added a comment.
Herald added a subscriber: ldionne.
@EricWF ping!
Repository:
rCXX libc++
https://reviews.llvm.org/D47344
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
This revision was automatically updated to reflect the committed changes.
Closed by commit rL336168: [Driver][Darwin] Use Host Triple to infer target os
version (authored by steven_wu, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews.llvm.org/D48849
Author: steven_wu
Date: Mon Jul 2 21:15:49 2018
New Revision: 336168
URL: http://llvm.org/viewvc/llvm-project?rev=336168=rev
Log:
[Driver][Darwin] Use Host Triple to infer target os version
Summary:
When clang required to infer target os version from --target option and
the os version is not
This revision was not accepted when it landed; it landed in state "Needs
Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit rCXX336165: [libc++] Install the missing header __errc
(authored by lichray, committed by ).
Changed prior to commit:
Author: lichray
Date: Mon Jul 2 21:01:44 2018
New Revision: 336165
URL: http://llvm.org/viewvc/llvm-project?rev=336165=rev
Log:
[libc++] Install the missing header __errc
Summary: Omitted from D41347.
Reviewers: EricWF
Subscribers: mgorny, christof, ldionne, cfe-commits
Differential
lichray created this revision.
Herald added subscribers: cfe-commits, ldionne, christof, mgorny.
Herald added a reviewer: EricWF.
Omitted from https://reviews.llvm.org/D41347.
Repository:
rCXX libc++
https://reviews.llvm.org/D48864
Files:
include/CMakeLists.txt
Index:
This revision was automatically updated to reflect the committed changes.
Closed by commit rCXX336164: [libc++] Lift std::errc into a separated header
(authored by lichray, committed by ).
Herald added a subscriber: ldionne.
Changed prior to commit:
Author: lichray
Date: Mon Jul 2 20:25:10 2018
New Revision: 336164
URL: http://llvm.org/viewvc/llvm-project?rev=336164=rev
Log:
[libc++] Lift std::errc into a separated header
Summary: This is needed to implement ``, otherwise `` would
need to include ``, which pulls in `` -- a header which
modocache created this revision.
modocache added reviewers: GorNishanov, EricWF.
If a user defines a coroutine_traits type that takes an incorrect
number of template parameters, or for some reason they include such
a type in their program, they receive a cryptic error message:
"too few template
mgrang updated this revision to Diff 153845.
https://reviews.llvm.org/D48862
Files:
lib/Driver/ToolChains/Gnu.cpp
lib/Driver/ToolChains/Linux.cpp
test/Driver/Inputs/openembedded_aarch64_linux_tree/usr/lib64/aarch64-oe-linux/6.3.0/crtbegin.o
juliehockett added a comment.
Please run clang-format and clang-tidy on the patch.
Comment at: clang-tools-extra/clang-doc/ClangDoc.cpp:25
#include "clang/Frontend/FrontendActions.h"
+#include "Representation.h"
Sort includes (this should be at the top)
mgrang created this revision.
mgrang added reviewers: dlj, rengolin, fedor.sergeev.
Herald added a reviewer: javed.absar.
Herald added a subscriber: kristof.beyls.
The lib paths are not correctly picked up for OpenEmbedded sysroots (like
arm-oe-linux-gnueabi) for 2 reasons:
1. OpenEmbedded
Author: epilk
Date: Mon Jul 2 18:30:53 2018
New Revision: 336162
URL: http://llvm.org/viewvc/llvm-project?rev=336162=rev
Log:
Revert r336159, r336157. Some bots failed on qualified std::max_align_t, and
other on unqualified max_align_t.
I'll take another stab at this tomorrow. Any ideas for
arphaman accepted this revision.
arphaman added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rC Clang
https://reviews.llvm.org/D48849
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
hfinkel added a comment.
In https://reviews.llvm.org/D48721#1150333, @bjope wrote:
> I tried running
>
> /clang -cc1 -O3 -funroll-loops -S -emit-llvm pragma-do-while-unroll.cpp -o
> - -mllvm -print-after-all
>
>
>
> and I get this
>
> ...
> !2 = distinct !{!2, !3}
> !3 =
steven_wu updated this revision to Diff 153836.
steven_wu added a comment.
It is easier and cleaner if I just fold everything into getOSVersion.
Repository:
rC Clang
https://reviews.llvm.org/D48849
Files:
lib/Driver/ToolChains/Darwin.cpp
test/Driver/clang-g-opts.c
Author: epilk
Date: Mon Jul 2 17:48:27 2018
New Revision: 336159
URL: http://llvm.org/viewvc/llvm-project?rev=336159=rev
Log:
Some buildbots were choking on std::max_align_t, try using the global alias.
Modified:
libcxxabi/trunk/src/cxa_demangle.cpp
Modified:
Author: epilk
Date: Mon Jul 2 17:23:18 2018
New Revision: 336157
URL: http://llvm.org/viewvc/llvm-project?rev=336157=rev
Log:
[demangler] Fix a MSVC alignment warning.
This should fix llvm.org/PR37944
Modified:
libcxxabi/trunk/src/cxa_demangle.cpp
Modified:
bjope added a comment.
I tried running
/clang -cc1 -O3 -funroll-loops -S -emit-llvm pragma-do-while-unroll.cpp -o -
-mllvm -print-after-all
and I get this
...
!2 = distinct !{!2, !3}
!3 = !{!"llvm.loop.unroll.count", i32 3}
!4 = !{!5, !5, i64 0}
!5 = !{!"int", !6, i64 0}
!6 =
Hello everyone,
Below are some buildbot numbers for the last week of 6/24/2018 - 6/30/2018.
Please see the same data in attached csv files:
The longest time each builder was red during the week;
"Status change ratio" by active builder (percent of builds that changed the
builder status from
Hello everyone,
Below are some buildbot numbers for the week of 6/17/2018 - 6/23/2018.
Please see the same data in attached csv files:
The longest time each builder was red during the week;
"Status change ratio" by active builder (percent of builds that changed the
builder status from greed to
arphaman added a comment.
Nit: Could you please try to extract the shared code into a function, e.g.
Optional overrideMacOSTripleDefaultVersion(const llvm::Triple
, ... OSTy, ... TheDriver) {
if (Triple.getOSMajorVersion())
return None;
llvm::Triple
erik.pilkington added a comment.
In https://reviews.llvm.org/D48322#1148424, @rsmith wrote:
> Hmm, so this will mean that we can have internal linkage declarations marked
> `Used` for which there is no definition, and we need to not warn on that.
>
> I think it might be better to avoid marking
NoQ added a comment.
Uhm, so we had an alpha checker enabled all along? Thanks for patching this up!
Comment at: lib/StaticAnalyzer/Checkers/CStringChecker.cpp:308
// These checks are either enabled by the CString out-of-bounds checker
-// explicitly or the "basic"
NoQ added inline comments.
Comment at: lib/StaticAnalyzer/Checkers/BlockInCriticalSectionChecker.cpp:68
: IILockGuard(nullptr), IIUniqueLock(nullptr),
- LockFn("lock"), UnlockFn("unlock"), SleepFn("sleep"), GetcFn("getc"),
- FgetsFn("fgets"), ReadFn("read"),
Author: rsmith
Date: Mon Jul 2 16:25:22 2018
New Revision: 336153
URL: http://llvm.org/viewvc/llvm-project?rev=336153=rev
Log:
Per C++ [over.match.copy]p1, direct-initialization of a reference can
only invoke converting constructors of the reference's underlying type.
Modified:
anniecherk updated this revision to Diff 153825.
anniecherk marked 4 inline comments as done.
anniecherk added a comment.
Addressed Julie's comments
https://reviews.llvm.org/D48395
Files:
clang-tools-extra/clang-doc/ClangDoc.cpp
clang-tools-extra/clang-doc/ClangDoc.h
steven_wu updated this revision to Diff 153822.
steven_wu added a comment.
handle *-apple-macosx target option
Repository:
rC Clang
https://reviews.llvm.org/D48849
Files:
lib/Driver/ToolChains/Darwin.cpp
test/Driver/clang-g-opts.c
test/Driver/target-triple-deployment.c
Index:
leonardchan added inline comments.
Comment at: include/clang/Basic/FixedPoint.h:31
+ SatNoPadding,
+};
+
ebevhan wrote:
> rjmccall wrote:
> > I figured you'd want this to be a struct which include the scale, width,
> > signed-ness, and saturating-ness; and
mclow.lists closed this revision.
mclow.lists added a comment.
Committed as revision 336132.
https://reviews.llvm.org/D48616
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
leonardchan updated this revision to Diff 153821.
leonardchan marked 10 inline comments as done.
leonardchan edited the summary of this revision.
leonardchan added a comment.
- Added tests
- Moved all conversion logic into `convert`
- Saturation is checked by checking the bits above the sign bit
steven_wu added a comment.
In https://reviews.llvm.org/D48849#1150246, @arphaman wrote:
> Hmm, the driver should not call `inferDeploymentTargetFromArch` when
> `-target` is passed. Or am I missing something?
Good call. This only handles the *-apple-darwin case. Maybe the same should
happen
arphaman added a comment.
Hmm, the driver should not call `inferDeploymentTargetFromArch` when `-target`
is passed. Or am I missing something?
Repository:
rC Clang
https://reviews.llvm.org/D48849
___
cfe-commits mailing list
steven_wu updated this revision to Diff 153816.
steven_wu added a comment.
Rebase the commit correctly
Repository:
rC Clang
https://reviews.llvm.org/D48849
Files:
lib/Driver/ToolChains/Darwin.cpp
test/Driver/clang-g-opts.c
Index: test/Driver/clang-g-opts.c
Wawha updated this revision to Diff 153813.
Wawha added a comment.
Here the third version to manage break with lambda in Allman.
I implement the modification propose by klimek.
If a lambda is detected, the BlockParameterCount and ParameterCount are
increment one more time in order to avoid extra
steven_wu updated this revision to Diff 153814.
steven_wu added a comment.
Update patch. Use a better API.
Repository:
rC Clang
https://reviews.llvm.org/D48849
Files:
lib/Driver/ToolChains/Darwin.cpp
test/Driver/clang-g-opts.c
Index: test/Driver/clang-g-opts.c
shuaiwang created this revision.
Herald added a reviewer: george.karpenkov.
Herald added subscribers: cfe-commits, a.sidorin.
This gives better coverage to the check as ExprMutationAnalyzer is more
accurate comparing to isOnlyUsedAsConst.
Majority of wins come from const usage of member field,
ahatanak updated this revision to Diff 153779.
ahatanak added a comment.
Herald added a subscriber: dexonsmith.
Implement the new rule defined here: http://wg21.link/p0968r0#2227. Produce a
diagnostic if a class is initialized with aggregate initialization and one of
its element's destructor is
Meinersbur updated this revision to Diff 153803.
Meinersbur added a comment.
- Adapt test file message.
Repository:
rC Clang
https://reviews.llvm.org/D48808
Files:
lib/CodeGen/CGLoopInfo.cpp
test/CodeGenCXX/pragma-loop-safety-nested.cpp
test/CodeGenCXX/pragma-loop-safety-outer.cpp
arphaman created this revision.
arphaman added a reviewer: jfb.
Herald added a subscriber: dexonsmith.
The '%tu'/'%td' as formatting specifiers have been used to print out the
NSInteger/NSUInteger values for a long time. Typically their ABI matches (i.e.
ptrdiff_t = NSInteger), but that's not
juliehockett added a reviewer: ioeric.
juliehockett added inline comments.
Comment at: clang-tools-extra/clang-doc/ClangDoc.h:27
+struct ClangDocContext {
+tooling::ExecutionContext *ECtx;
Can we put this in `Representation.h`? Here, you have a potential
steven_wu added a comment.
Unfortunately, I wasn't able to write a test for this because the host triple
in the configuration can either be x86_64-apple-darwin* or
x86_64-apple-macosx*, but the one used passed by driver is always macosx one. I
can't reliably compare those two.
Repository:
steven_wu created this revision.
steven_wu added reviewers: arphaman, dexonsmith.
When clang required to infer target os version from --target option and
the os version is not specified in targets, check the host triple. If the
host and target are both macOS, use host triple to infer target os
NoQ added inline comments.
Comment at: include/clang/Analysis/ConstructionContext.h:90
assert(Other);
return (Trigger == Other->Trigger);
}
Uhm. Will fix.
Repository:
rC Clang
https://reviews.llvm.org/D48681
juliehockett added inline comments.
Comment at: clang-tools-extra/clang-doc/tool/ClangDocMain.cpp:249
+if (EnclosingScope.get().RefType == doc::InfoType::IT_function)
+ continue;
+
ioeric wrote:
> Are symbols declared in functions indexed by the
anniecherk updated this revision to Diff 153790.
anniecherk added a comment.
Updated the tests to more accurately check that the files that we expect to not
be generated by clang-doc with the public flag are in fact not being created.
The tests now run clang-doc over the test files with and
juliehockett updated this revision to Diff 153787.
juliehockett marked 12 inline comments as done.
juliehockett added a comment.
Refactoring second reduce code into the library, and adding bitcode
reader/writer support
https://reviews.llvm.org/D48341
Files:
craig.topper updated this revision to Diff 153786.
craig.topper marked 4 inline comments as done.
craig.topper added a comment.
-Added a negative test
-Hopefully fixed all the grammatical/spelling errors.
-Attempted to clarify some more about prefer-vector-width and builtins.
aaron.ballman added inline comments.
Comment at: include/clang/Basic/Attr.td:1949
+ let Args = [UnsignedArgument<"VectorWidth">];
+ let Subjects = SubjectList<[Function], ErrorDiag>;
+ let Documentation = [Undocumented];
chandlerc wrote:
> aaron.ballman
chandlerc added a comment.
FWIW, I looked at an early version of this patch and am generally happy with
the target-specific / IR-specific behavior aspects of it. Totally leaving the
detailed review of the attribute stuff to you Aaron, as you're already doing an
amazing job there. Minor
Author: pirama
Date: Mon Jul 2 13:11:15 2018
New Revision: 336141
URL: http://llvm.org/viewvc/llvm-project?rev=336141=rev
Log:
[Win32] Overload ==, != for locale_t and long long
Summary:
_is_chartype_l (needed for isxdigit_l) in MinGW compares locale_t and NULL.
NULL is 'long long' for 64-bit,
This revision was automatically updated to reflect the committed changes.
Closed by commit rCXX336141: [Win32] Overload ==, != for locale_t and long long
(authored by pirama, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D48749?vs=153411=153778#toc
Repository:
rCXX
j10kay updated this revision to Diff 153773.
j10kay marked 3 inline comments as done.
j10kay added a comment.
Implemented the code review changes
Repository:
rL LLVM
https://reviews.llvm.org/D48601
Files:
clang/include/clang/Driver/Options.td
clang/lib/Driver/Driver.cpp
Quuxplusone added inline comments.
Comment at: libcxx/include/memory:1479
+struct __has_construct_missing
+: false_type
+{
vsapsai wrote:
> vsapsai wrote:
> > vsapsai wrote:
> > > erik.pilkington wrote:
> > > > vsapsai wrote:
> > > > > erik.pilkington wrote:
aaron.ballman added inline comments.
Comment at: include/clang/Basic/Attr.td:1949
+ let Args = [UnsignedArgument<"VectorWidth">];
+ let Subjects = SubjectList<[Function], ErrorDiag>;
+ let Documentation = [Undocumented];
craig.topper wrote:
> aaron.ballman
Meinersbur updated this revision to Diff 153769.
Meinersbur added a comment.
- Allow multiple parallel annotations
Repository:
rC Clang
https://reviews.llvm.org/D48808
Files:
lib/CodeGen/CGLoopInfo.cpp
test/CodeGenCXX/pragma-loop-safety-nested.cpp
This revision was not accepted when it landed; it landed in state "Needs
Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit rC336137: [CodeGenCXX] Emit strip.invariant.group with
-fstrict-vtable-pointers (authored by Prazek, committed by ).
Author: prazek
Date: Mon Jul 2 12:21:36 2018
New Revision: 336137
URL: http://llvm.org/viewvc/llvm-project?rev=336137=rev
Log:
[CodeGenCXX] Emit strip.invariant.group with -fstrict-vtable-pointers
Summary:
Emmiting new intrinsic that strips invariant.groups to make
devirtulization sound, as
vsapsai planned changes to this revision.
vsapsai added inline comments.
Comment at: libcxx/include/memory:1479
+struct __has_construct_missing
+: false_type
+{
vsapsai wrote:
> vsapsai wrote:
> > erik.pilkington wrote:
> > > vsapsai wrote:
> > > >
vsapsai added a comment.
In https://reviews.llvm.org/D48342#1148751, @mclow.lists wrote:
> I want to point out that this code (not -necessarily- this patch, but where
> it lives) needs to be rewritten.
>
> There is no prohibition on users specializing `allocator_traits` for their
> allocators,
aaron.ballman added inline comments.
Comment at: test/Sema/attr-min-vector-width.c:8
+
+void f3(void) __attribute__((__min_vector_width__(128),
__min_vector_width__(256))); /* expected-warning {{attribute
'__min_vector_width__' is already applied with different parameters}} */
craig.topper added inline comments.
Comment at: include/clang/Basic/Attr.td:1949
+ let Args = [UnsignedArgument<"VectorWidth">];
+ let Subjects = SubjectList<[Function], ErrorDiag>;
+ let Documentation = [Undocumented];
aaron.ballman wrote:
> Should this
craig.topper updated this revision to Diff 153763.
craig.topper added a comment.
-Rebase the intrinsic headers and builtins file
-Add documentation for the attribute. Open to feedback on improvements here
-Add tests for wrong number of arguments to the attribute.
https://reviews.llvm.org/D48617
craig.topper added inline comments.
Comment at: test/Sema/attr-min-vector-width.c:8
+
+void f3(void) __attribute__((__min_vector_width__(128),
__min_vector_width__(256))); /* expected-warning {{attribute
'__min_vector_width__' is already applied with different parameters}} */
rjmccall added inline comments.
Comment at: clang/lib/CodeGen/CGExprScalar.cpp:1647
+ }
+}
+
Prazek wrote:
> rsmith wrote:
> > Prazek wrote:
> > > rjmccall wrote:
> > > > Prazek wrote:
> > > > > rjmccall wrote:
> > > > > > Incidentally, how do you
Author: marshall
Date: Mon Jul 2 11:41:15 2018
New Revision: 336132
URL: http://llvm.org/viewvc/llvm-project?rev=336132=rev
Log:
Implement LWG 2946, 3075 and 3076. Reviewed as https://reviews.llvm.org/D48616
Added:
hfinkel added a comment.
In https://reviews.llvm.org/D48808#1149862, @Meinersbur wrote:
> In https://reviews.llvm.org/D48808#1149845, @hfinkel wrote:
>
> > We specifically defined the metadata to support nested loops. The LangRef
> > says, "The llvm.mem.parallel_loop_access metadata refers to a
Meinersbur added a comment.
In https://reviews.llvm.org/D48808#1149845, @hfinkel wrote:
> We specifically defined the metadata to support nested loops. The LangRef
> says, "The llvm.mem.parallel_loop_access metadata refers to a loop
> identifier, or metadata containing a list of loop
On 07/02/2018 12:27 PM, Tyler Nowicki wrote:
> Hi Michael, Hal,
>
> Sorry it has been a while since I looked at this. My memory is a
> little fuzzy. The intent of 'assume_safety' is to tell LAA to
> skip dependency checking on loads and stores so the vectorizer doesn't
> stop as soon as it sees
hfinkel added a comment.
In https://reviews.llvm.org/D48808#1149828, @Meinersbur wrote:
> In https://reviews.llvm.org/D48808#1149549, @hfinkel wrote:
>
> > In https://reviews.llvm.org/D48808#1149534, @ABataev wrote:
> >
> > >
> >
> >
> > Michael, can you please add a test with two inner loops,
a.sidorin added a comment.
Whoops, sorry Balázs. Didn't look at the review author :(
Repository:
rC Clang
https://reviews.llvm.org/D48722
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
a.sidorin added a comment.
Hello Gabor,
I have a strong feeling of duplication with attribute and flags merging move in
https://reviews.llvm.org/D47632. Maybe it is better to be resolved in that
review by using the same code for attr/flag merging for both newly-created and
mapped decls?
Meinersbur added a comment.
In https://reviews.llvm.org/D48808#1149549, @hfinkel wrote:
> In https://reviews.llvm.org/D48808#1149534, @ABataev wrote:
>
> >
>
>
> Michael, can you please add a test with two inner loops, one where more than
> one is annotated, and one where only the outer loop is
Hi Michael, Hal,
Sorry it has been a while since I looked at this. My memory is a little
fuzzy. The intent of 'assume_safety' is to tell LAA to skip dependency
checking on loads and stores so the vectorizer doesn't stop as soon as it
sees both in a loop. At the time 'assume_safety' was
Author: george.karpenkov
Date: Mon Jul 2 10:10:40 2018
New Revision: 336124
URL: http://llvm.org/viewvc/llvm-project?rev=336124=rev
Log:
[analyzer] [tests] Pass clang executable path to prefix-less executor scripts.
Modified:
cfe/trunk/utils/analyzer/SATestBuild.py
Modified:
jmorse added a comment.
Ping
Repository:
rC Clang
https://reviews.llvm.org/D48072
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
benhamilton accepted this revision.
benhamilton added a comment.
This revision is now accepted and ready to land.
> Sorry for the confusion. If you prefer me to add this test and modify in
> later commit I'll do it.
I see. Just mentioning what you did in the diff description is probably OK.
benhamilton accepted this revision.
benhamilton added a comment.
This revision is now accepted and ready to land.
In https://reviews.llvm.org/D48716#1149293, @jolesiak wrote:
> General comment to changes https://reviews.llvm.org/D48716,
> https://reviews.llvm.org/D48718,
NoQ added a comment.
That'd be a hell for you because when the container is updated you won't be
able to easily find iterators all that iterate over it. Normally what you want
to do is keep mapping iterators to container regions, and when the region dies,
"freeze" the data (make sure it can no
malaperle updated this revision to Diff 153742.
malaperle added a comment.
Rebased.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D47846
Files:
clangd/ClangdLSPServer.cpp
clangd/ClangdLSPServer.h
clangd/ClangdServer.cpp
clangd/ClangdServer.h
clangd/FindSymbols.cpp
This revision was automatically updated to reflect the committed changes.
Closed by commit rL336119: [clangd] Implement hover for auto and
decltype (authored by malaperle, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews.llvm.org/D48159
Files:
Author: malaperle
Date: Mon Jul 2 09:28:34 2018
New Revision: 336119
URL: http://llvm.org/viewvc/llvm-project?rev=336119=rev
Log:
[clangd] Implement hover for "auto" and "decltype"
Summary:
This allows hovering on keywords that refer to deduced types.
This should cover most useful cases. Not
malaperle added inline comments.
Comment at: clangd/XRefs.cpp:559
+ //- auto& i = 1;
+ bool VisitDeclaratorDecl(DeclaratorDecl *D) {
+if (!D->getTypeSourceInfo() ||
klimek wrote:
> klimek wrote:
> > malaperle wrote:
> > > klimek wrote:
> > > > sammccall
malaperle updated this revision to Diff 153737.
malaperle added a comment.
Tweak comment with FIXME.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D48159
Files:
clangd/XRefs.cpp
unittests/clangd/TestTU.cpp
unittests/clangd/TestTU.h
unittests/clangd/XRefsTests.cpp
cryptoad created this revision.
cryptoad added reviewers: eugenis, alekseyshl.
Scudo works on PPC64 as is, so mark the architecture as supported for it. This
will also require a change to config-ix.cmake on the compiler-rt side.
Update the tests accordingly.
Repository:
rC Clang
Meinersbur added a comment.
In https://reviews.llvm.org/D48808#1149534, @ABataev wrote:
> I don't think that this is the intended behavior of the `#pragma clang loop`.
> it is better to ask the author of this pragma is this correct or not.
I understand it as the intended behavior of the
hfinkel added a comment.
In https://reviews.llvm.org/D48808#1149534, @ABataev wrote:
> I don't think that this is the intended behavior of the `#pragma clang loop`.
> it is better to ask the author of this pragma is this correct or not.
It is the intended behavior that the memory accesses are
ABataev added a comment.
I don't think that this is the intended behavior of the `#pragma clang loop`.
it is better to ask the author of this pragma is this correct or not.
Repository:
rC Clang
https://reviews.llvm.org/D48808
___
cfe-commits
Hi Richard,
This introduced test failures on Windows, but for some reason only in
32-bit builds: https://bugs.llvm.org/show_bug.cgi?id=38015
I don't know what's going on yet, just finished the bisection. Maybe
it reproduces in 32-bit linux builds too?
- Hans
On Fri, Jun 29, 2018 at 11:58 PM,
Author: balazske
Date: Mon Jul 2 07:14:07 2018
New Revision: 336108
URL: http://llvm.org/viewvc/llvm-project?rev=336108=rev
Log:
Test commit access
Modified:
cfe/trunk/unittests/AST/ASTImporterTest.cpp
Modified: cfe/trunk/unittests/AST/ASTImporterTest.cpp
URL:
krytarowski added a comment.
ping^3
Repository:
rL LLVM
https://reviews.llvm.org/D47814
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jolesiak marked an inline comment as done.
jolesiak added inline comments.
Comment at: lib/Format/ContinuationIndenter.cpp:1411
+ // line).
+ if (Current.MatchingParen && Current.MatchingParen->Previous) {
+const FormatToken = *Current.MatchingParen->Previous;
jolesiak updated this revision to Diff 153706.
jolesiak added a comment.
Rebase.
Repository:
rC Clang
https://reviews.llvm.org/D48720
Files:
lib/Format/ContinuationIndenter.cpp
unittests/Format/FormatTestObjC.cpp
Index: unittests/Format/FormatTestObjC.cpp
jolesiak updated this revision to Diff 153705.
jolesiak added a comment.
Improve condition.
Repository:
rC Clang
https://reviews.llvm.org/D48720
Files:
lib/Format/ContinuationIndenter.cpp
lib/Format/FormatToken.h
lib/Format/TokenAnnotator.cpp
unittests/Format/FormatTestObjC.cpp
kosarev created this revision.
kosarev added reviewers: SjoerdMeijer, jgreenhalgh, rengolin.
kosarev added a project: clang.
Herald added a reviewer: javed.absar.
Herald added a subscriber: kristof.beyls.
This patch adds support for vrndi_f32() and vrndiq_f32() intrinsics in AArch32
mode and for
baloghadamsoftware added a comment.
Hmm, then the only solution that comes to my mind is to link iterator positions
to container data instead of the container regions. I also have to store a link
to the class definition of the container in the container data for the
invalidation rules.
baloghadamsoftware updated this revision to Diff 153698.
baloghadamsoftware added a comment.
Updated according to the comments and assertions added to fail the tests
without the fix.
https://reviews.llvm.org/D48764
Files:
lib/StaticAnalyzer/Checkers/IteratorChecker.cpp
Index:
thakis added a comment.
Thanks, landed with nit addressed in r336097.
https://reviews.llvm.org/D48781
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: nico
Date: Mon Jul 2 05:31:20 2018
New Revision: 336097
URL: http://llvm.org/viewvc/llvm-project?rev=336097=rev
Log:
[ms] Fix mangling of char16_t and char32_t to be compatible with MSVC.
MSVC limits char16_t and char32_t string literal names to 32 bytes of character
data, not to 32
hans accepted this revision.
hans added a comment.
This revision is now accepted and ready to land.
lgtm with nit
Comment at: clang/lib/AST/MicrosoftMangle.cpp:3262
+ // Enforce our 32 bytes max, except wchar_t which gets 32 chars instead.
+ unsigned MaxBytesToWrite =
1 - 100 of 115 matches
Mail list logo
