benhamilton requested changes to this revision.
benhamilton added a comment.
This revision now requires changes to proceed.
> Count selector parts also for method declarations.
What bug does this fix? Can you add a test which breaks before this change and
is fixed by this change?
benhamilton requested changes to this revision.
benhamilton added a comment.
This revision now requires changes to proceed.
Can you add a test, please?
Repository:
rC Clang
https://reviews.llvm.org/D48718
___
cfe-commits mailing list
cfe-commits@
simark created this revision.
Herald added subscribers: cfe-commits, jkorous, ioeric, ilya-biryukov.
This is not intended to be merged, it is my attempt at reproducing this:
https://bugs.llvm.org/show_bug.cgi?id=37963
using a unit test. However, I can't manage to get the faulty result
like this
v.g.vassilev added a comment.
It looks like the diagnostic options and few others were moved out from the
control block in r297655 by Manman Ren. The header files are still part of the
control block. The best I can do is buffer that information and print it out
last. Is that what we want?
Rep
simark added a comment.
FYI, I have uploaded my attempt at writing a unit test here:
https://reviews.llvm.org/D48726
I tried to make it as close as possible to the example in the bug report, but
don't manage to reproduce the bug.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/
leonardchan created this revision.
leonardchan added reviewers: phosek, mcgrathr, jakehehrlich, rjmccall, ebevhan,
rsmith.
leonardchan added a project: clang.
- Rename the `-fsame-fbits` flag to `-fpadding-on-unsigned-fixed-point`
- Move the flag from a driver option to a cc1 option
- Rename the
leonardchan added inline comments.
Comment at: include/clang/Basic/TargetInfo.h:90
+ // by default.
+ bool SameFBits;
+
rjmccall wrote:
> Sorry for the extremely late review, but this really needs to be renamed.
> Please remember that other compiler maintaine
leonardchan updated this revision to Diff 153335.
leonardchan added a comment.
Formatting and forgot to get rid of `Group` and `Flags`
Repository:
rC Clang
https://reviews.llvm.org/D48727
Files:
include/clang/Basic/LangOptions.def
include/clang/Basic/TargetInfo.h
include/clang/Driver/C
rjmccall added inline comments.
Comment at: include/clang/Basic/FixedPoint.h:23
+
+class FixedPointNumber {
+ public:
ebevhan wrote:
> rjmccall wrote:
> > The established naming convention here — as seen in `APInt`, `APFloat`,
> > `APValue`, etc. — would call th
rjmccall added a comment.
This all looks reasonable except that I think the interpretation is exactly
backwards, no? From the documentation on the option,
`-fpadding-on-unsigned-fixed-point` causes there to be padding, i.e. the
inverse of the old `SameFBits`; but the default value, comments, a
bruno accepted this revision.
bruno added a comment.
This revision is now accepted and ready to land.
> It looks like the diagnostic options and few others were moved out from the
> control block in r297655 by Manman Ren. The header files are still part of
> the control block. The best I can d
leonardchan added a comment.
Oh, having the same number of fractional bits is what leads to unsigned types
having one bit of padding, and vice versa.
If this flag is set to false, then the integral and fractional parts of the
unsigned types take up the whole bit width of the underlying scaled i
rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
In https://reviews.llvm.org/D48727#1146873, @leonardchan wrote:
> Oh, having the same number of fractional bits is what leads to unsigned types
> having one bit of padding, and vice versa.
Author: ioeric
Date: Thu Jun 28 09:51:12 2018
New Revision: 335874
URL: http://llvm.org/viewvc/llvm-project?rev=335874&view=rev
Log:
[clangd] Use log10 instead of the natural logrithm for usage boost.
Modified:
clang-tools-extra/trunk/clangd/Quality.cpp
clang-tools-extra/trunk/unittests/c
Sorry for the breakage. All issues should be fixed by now.
On Thu, Jun 28, 2018 at 8:26 AM Mikael Holmén
wrote:
> Hi,
>
> I get the same compilation error when I compile with clang 3.6.0.
>
> Revert?
>
> Regards,
> Mikael
>
> On 06/28/2018 03:16 AM, via cfe-commits wrote:
> > Hi, this commit al
krasimir created this revision.
Herald added a subscriber: cfe-commits.
This stops clang-format from touching raw string contents with unrecognized
delimiters inside recognized functions.
Unrecognized delimiters signal that the string might be special.
Repository:
rC Clang
https://reviews.ll
Author: krasimir
Date: Thu Jun 28 09:58:24 2018
New Revision: 335876
URL: http://llvm.org/viewvc/llvm-project?rev=335876&view=rev
Log:
[clang-format] Do not format raw string literals inside a recognized function
with a non-recognized delimiter
Summary:
This stops clang-format from touching raw
JonasToth marked 3 inline comments as done.
JonasToth added inline comments.
Comment at: clang-tidy/cppcoreguidelines/ProBoundsPointerArithmeticCheck.cpp:24
+ const auto AllPointerTypes = anyOf(
+ hasType(pointerType()),
hasType(autoType(hasDeducedType(pointerType();
JonasToth updated this revision to Diff 153341.
JonasToth added a comment.
- address review comments
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D48717
Files:
clang-tidy/cppcoreguidelines/ProBoundsPointerArithmeticCheck.cpp
test/clang-tidy/cppcoreguidelines-pro-bounds-poi
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 rL335876: [clang-format] Do not format raw string literals
inside a recognized function… (authored by krasimir, committed by
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/D48717
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
JonasToth marked an inline comment as not done.
JonasToth added inline comments.
Comment at:
test/clang-tidy/cppcoreguidelines-pro-bounds-pointer-arithmetic-pr36489.cpp:4
+
+// Fix PR36489 and detect auto-deduced value correctly.
+char *getPtr();
aaron.ballman w
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM with a small documentation nit.
Comment at: docs/clang-tidy/checks/cert-msc51-cpp.rst:39
+
+ A comma-separated list of the type names which are disallowe
craig.topper added inline comments.
Comment at: clang/lib/Headers/avx512vlintrin.h:7421
+ __builtin_convertvector((__v4si)__A, __v4qi), (__v4qi){0, 0, 0, 0}, 0, 1,
+ 2, 3, 4, 5, 6, 7, 7, 7, 7, 7, 7, 7, 7, 7);
}
If you need to add more zeroes than the w
Author: tra
Date: Thu Jun 28 10:15:52 2018
New Revision: 335880
URL: http://llvm.org/viewvc/llvm-project?rev=335880&view=rev
Log:
[CUDA] Place all CUDA sections in __NV_CUDA segment on Mac.
That's where CUDA binaries appear to put them.
Differential Revision: https://reviews.llvm.org/D48615
Mod
This revision was automatically updated to reflect the committed changes.
Closed by commit rL335880: [CUDA] Place all CUDA sections in __NV_CUDA segment
on Mac. (authored by tra, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D48615?vs
pirama created this revision.
pirama added reviewers: compnerd, srhines, danalbert, mstorsjo.
Herald added subscribers: cfe-commits, ldionne, christof.
Use _LIBCPP_MSVCRT_LIKE while configuring ELAST, so MinGW gets the same
configuration as MSVC.
Repository:
rCXX libc++
https://reviews.llvm.o
RKeelan added a comment.
My team is waiting for this feature, too.
Repository:
rL LLVM
https://reviews.llvm.org/D28462
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: vedantk
Date: Thu Jun 28 10:53:35 2018
New Revision: 335890
URL: http://llvm.org/viewvc/llvm-project?rev=335890&view=rev
Log:
[Darwin] Remove _VPMergeHook from the auto-export list
Remove _VPMergeHook from Darwin's automatically-exported symbol list for
PGO. As of r328987 this symbol is n
bruno accepted this revision.
bruno added a comment.
This revision is now accepted and ready to land.
Thanks for working on this, LGTM
https://reviews.llvm.org/D48367
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-
Author: aaronballman
Date: Thu Jun 28 11:03:00 2018
New Revision: 335896
URL: http://llvm.org/viewvc/llvm-project?rev=335896&view=rev
Log:
Set the line ending of clang.natvis to CRLF.
This corresponds to r335889, which does the same thing for llvm.natvis.
Modified:
cfe/trunk/utils/ClangVisua
ldionne added inline comments.
Comment at: include/string:836
_LIBCPP_METHOD_TEMPLATE_IMPLICIT_INSTANTIATION_VIS
-basic_string(const _Tp& __t, size_type __pos, size_type __n,
- const allocator_type& __a = allocator_type(),
-
rsmith added inline comments.
Comment at: clang/lib/CodeGen/CGExprScalar.cpp:1647
+ }
+}
+
Prazek wrote:
> rjmccall wrote:
> > Prazek wrote:
> > > rjmccall wrote:
> > > > Incidentally, how do you protect against code like this?
> > > >
> > > > A *ptr;
enyquist added a comment.
@klimek fair point. To be honest, I've pretty much lost interest / momentum on
this feature, I very much doubt I will ever go back and re-implement from
scratch as you suggest.
Not meaning to sound rude, I just don't want to waste anyone's time who is
waiting for this
GBuella added inline comments.
Comment at: test/CodeGen/avx-builtins.c:1423
-
-__m256 test_mm256_cmp_ps_true(__m256 a, __m256 b) {
- // CHECK-LABEL: @test_mm256_cmp_ps_true
spatel wrote:
> GBuella wrote:
> > spatel wrote:
> > > Why are we deleting tests instead
Author: ab
Date: Thu Jun 28 11:35:25 2018
New Revision: 335898
URL: http://llvm.org/viewvc/llvm-project?rev=335898&view=rev
Log:
[CMake] Make bootstrap and compiler-rt depend on cxx-headers.
Since r334468, we no longer always copy the libcxx headers by virtue of
their presence when cmake runs.
T
Author: ab
Date: Thu Jun 28 11:35:35 2018
New Revision: 335899
URL: http://llvm.org/viewvc/llvm-project?rev=335899&view=rev
Log:
[CMake] Rename cxx_headers back to cxx-headers.
r334477 renamed the cxx-headers target to cxx_headers, but various
pieces sort-of expect the target names to match the c
This revision was automatically updated to reflect the committed changes.
Closed by commit rL335898: [CMake] Make bootstrap and compiler-rt depend on
cxx-headers. (authored by ab, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D48700?v
This revision was automatically updated to reflect the committed changes.
Closed by commit rL335899: [CMake] Rename cxx_headers back to cxx-headers.
(authored by ab, committed by ).
Herald added subscribers: llvm-commits, christof.
Changed prior to commit:
https://reviews.llvm.org/D48701?vs=153
rupprecht created this revision.
rupprecht added a reviewer: saugustine.
Herald added subscribers: cfe-commits, chrib, christof.
Repository:
rUNW libunwind
https://reviews.llvm.org/D48733
Files:
src/AddressSpace.hpp
Index: src/AddressSpace.hpp
==
Meinersbur created this revision.
Meinersbur added reviewers: dlj, rsmith, Jean-Daniel.
If a function has multiple format_arg attributes, clang only considers the
first it finds (because AttributeLists are in reverse order, it is textual
last) and ignores all others.
Loop over all FormatArgAttr
Meinersbur added inline comments.
Comment at: lib/Sema/SemaChecking.cpp:5534
+ if (!IsFirst)
+return CommonResult;
+
Not sure what should be returned here; To minimize surprises, this returns what
the current version would have returned.
Repositor
jdenny created this revision.
jdenny added a reviewer: ABataev.
Herald added a subscriber: guansong.
A reduction for an incomplete array type used to produce an assert
fail during codegen. Now it produces a diagnostic.
Repository:
rC Clang
https://reviews.llvm.org/D48735
Files:
clang/lib/
ABataev accepted this revision.
ABataev added a comment.
This revision is now accepted and ready to land.
LG
Repository:
rC Clang
https://reviews.llvm.org/D48735
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bi
craig.topper added inline comments.
Comment at: clang/lib/Headers/avx512vlintrin.h:33
+typedef short __v2hi __attribute__((__vector_size__(4)));
+typedef char __v4qi __attribute__((__vector_size__(4)));
Can you just do a local typedef in the functions that need
arphaman created this revision.
arphaman added a reviewer: bruno.
Herald added a subscriber: dexonsmith.
The new C++ stdlib warning added in https://reviews.llvm.org/D48297 gets
triggered when compiling an assembly file with -x assembler-with-cpp.
This commit ensures that the C++ stdlib is not in
Author: jdenny
Date: Thu Jun 28 12:46:10 2018
New Revision: 335907
URL: http://llvm.org/viewvc/llvm-project?rev=335907&view=rev
Log:
[OPENMP] Fix incomplete type check for array reductions
A reduction for an incomplete array type used to produce an assert
fail during codegen. Now it produces a d
tcanens added inline comments.
Comment at: include/string:856
+_LIBCPP_METHOD_TEMPLATE_IMPLICIT_INSTANTIATION_VIS
+explicit basic_string(const _Tp& __t, const allocator_type& __a);
+
ldionne wrote:
> I think this `explicit` shouldn't be there, too
Author: jdenny
Date: Thu Jun 28 12:54:27 2018
New Revision: 335910
URL: http://llvm.org/viewvc/llvm-project?rev=335910&view=rev
Log:
Revert r335907: [OPENMP] Fix incomplete type check for array reductions
Sorry, forgot to add commit log attributes again.
Removed:
cfe/trunk/test/OpenMP/parall
This revision was automatically updated to reflect the committed changes.
Closed by commit rC335911: [OPENMP] Fix incomplete type check for array
reductions (authored by jdenny, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D48735?vs=153366&id=153374#toc
Repository:
rC Cl
Author: jdenny
Date: Thu Jun 28 12:54:49 2018
New Revision: 335911
URL: http://llvm.org/viewvc/llvm-project?rev=335911&view=rev
Log:
[OPENMP] Fix incomplete type check for array reductions
A reduction for an incomplete array type used to produce an assert
fail during codegen. Now it produces a d
mstorsjo accepted this revision.
mstorsjo added a comment.
LGTM, test built it and it seems to work fine.
Repository:
rCXX libc++
https://reviews.llvm.org/D48731
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bi
Author: pirama
Date: Thu Jun 28 13:16:45 2018
New Revision: 335916
URL: http://llvm.org/viewvc/llvm-project?rev=335916&view=rev
Log:
Configure ELAST for MinGW
Summary:
Use _LIBCPP_MSVCRT_LIKE while configuring ELAST, so MinGW gets the same
configuration as MSVC.
Reviewers: compnerd, srhines, dan
This revision was automatically updated to reflect the committed changes.
Closed by commit rCXX335916: Configure ELAST for MinGW (authored by pirama,
committed by ).
Changed prior to commit:
https://reviews.llvm.org/D48731?vs=153352&id=153381#toc
Repository:
rCXX libc++
https://reviews.llvm
juliehockett accepted this revision.
juliehockett added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rC Clang
https://reviews.llvm.org/D48710
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llv
Author: phosek
Date: Thu Jun 28 13:19:11 2018
New Revision: 335917
URL: http://llvm.org/viewvc/llvm-project?rev=335917&view=rev
Log:
[CMake] Build static runtimes for host in Fuchsia first stage
First stage build is only a minimal build where we don't need
a complete multiarch support, but we nee
This revision was automatically updated to reflect the committed changes.
Closed by commit rC335917: [CMake] Build static runtimes for host in Fuchsia
first stage (authored by phosek, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D48707?vs=153270&id=153383#toc
Repository:
mikhail.ramalho abandoned this revision.
mikhail.ramalho added a comment.
We won't need this patch anymore because we'll try another approach in
https://reviews.llvm.org/D48565.
Repository:
rC Clang
https://reviews.llvm.org/D48561
___
cfe-commit
Author: phosek
Date: Thu Jun 28 13:33:18 2018
New Revision: 335919
URL: http://llvm.org/viewvc/llvm-project?rev=335919&view=rev
Log:
[CMake] Don't build Linux targets on Darwin in Fuchsia toolchain
This is currently breaking because Linux target sysroots rely on
case sensitive filesystem which is
This revision was automatically updated to reflect the committed changes.
Closed by commit rC335919: [CMake] Don't build Linux targets on Darwin in
Fuchsia toolchain (authored by phosek, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D48710?vs=153275&id=153385#toc
Repository
malaperle added inline comments.
Comment at: clangd/XRefs.cpp:559
+ //- auto& i = 1;
+ bool VisitDeclaratorDecl(DeclaratorDecl *D) {
+if (!D->getTypeSourceInfo() ||
klimek wrote:
> sammccall wrote:
> > malaperle wrote:
> > > sammccall wrote:
> > > > out of
george.burgess.iv added a comment.
Ping :)
Repository:
rC Clang
https://reviews.llvm.org/D47840
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
> I'm not 100% thrilled that we're emitting two warnings about the same thing
> for slightly different reasons; alternatives welcome. :)
Me either, but given our policy that warning flags just
deepak2427 updated this revision to Diff 153392.
deepak2427 added a comment.
Herald added a subscriber: zzheng.
Add tests.
https://reviews.llvm.org/D48721
Files:
test/CodeGen/pragma-do-while-unroll.cpp
test/CodeGen/pragma-do-while.cpp
Index: test/CodeGen/pragma-do-while.cpp
==
deepak2427 updated this revision to Diff 153393.
deepak2427 added a comment.
Add tests and the patch.
https://reviews.llvm.org/D48721
Files:
lib/CodeGen/CGStmt.cpp
test/CodeGen/pragma-do-while-unroll.cpp
test/CodeGen/pragma-do-while.cpp
Index: test/CodeGen/pragma-do-while.cpp
===
Author: rsmith
Date: Thu Jun 28 14:17:55 2018
New Revision: 335925
URL: http://llvm.org/viewvc/llvm-project?rev=335925&view=rev
Log:
PR37979: integral promotions in C++ treat enum bit-fields like enums,
not like bit-fields.
We used to get this right "by accident", because conversions for the
sele
lebedev.ri added a comment.
I'm not sure we can use `-O` in tests at all, and i'm not sure it is even
needed here since you are only testing codegen.
Repository:
rC Clang
https://reviews.llvm.org/D48721
___
cfe-commits mailing list
cfe-commits@l
Author: mramalho
Date: Thu Jun 28 14:26:52 2018
New Revision: 335926
URL: http://llvm.org/viewvc/llvm-project?rev=335926&view=rev
Log:
[analyzer] Fix wrong comparison generation of the ranges generated by the
refutation manager
The refutation manager is removing a true bug from the test in this
pirama created this revision.
pirama added reviewers: EricWF, srhines, danalbert.
Herald added subscribers: ldionne, christof.
This function is passed as the __at_exit parameter to
__libcpp_tls_create. This parameter is marked with the
_LIBCPP_TLS_DESTRUCTOR_CC attribute. The macro is empty for
pirama created this revision.
pirama added reviewers: EricWF, srhines, danalbert.
Herald added subscribers: ldionne, christof.
Add the _LIBCPP_TLS_DESTRUCTOR_CC attribute to the __at_exit parameter
for the pthread definition of __libcpp_get_tls. This makes it
consistent with the rest (the non-pth
rsmith accepted this revision.
rsmith added inline comments.
This revision is now accepted and ready to land.
Comment at: llvm/docs/LangRef.rst:12928
+established by ``invariant.group`` metadata no longer holds, to obtain a new
pointer
+value that does carries fresh invariant gr
george.burgess.iv added a comment.
Thanks!
Repository:
rC Clang
https://reviews.llvm.org/D47840
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
This revision was automatically updated to reflect the committed changes.
Closed by commit rL335927: [Parse] Make -Wgcc-compat complain about for loop
inits in C89 (authored by gbiv, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D4784
Author: gbiv
Date: Thu Jun 28 14:36:00 2018
New Revision: 335927
URL: http://llvm.org/viewvc/llvm-project?rev=335927&view=rev
Log:
[Parse] Make -Wgcc-compat complain about for loop inits in C89
While clang allows declarations in for loop init statements in c89 and
gnu89, gcc does not. So, we shou
Author: mramalho
Date: Thu Jun 28 14:39:41 2018
New Revision: 335929
URL: http://llvm.org/viewvc/llvm-project?rev=335929&view=rev
Log:
[analyzer] Move test to the correct directory
It was accidentaly pushed in r335926
Added:
cfe/trunk/test/Analysis/PR37855.c
- copied, changed from r335
pirama created this revision.
pirama added reviewers: mstorsjo, EricWF, srhines, danalbert.
Herald added subscribers: ldionne, christof.
_is_chartype_l (needed for isxdigit_l) in MinGW compares locale_t and NULL.
NULL is 'long long' for 64-bit, and this results in ambiguous overloads when
compiled
bruno added inline comments.
Comment at: lib/Frontend/InitHeaderSearch.cpp:474
+ if (Lang.CPlusPlus && !Lang.AsmPreprocessor &&
+ HSOpts.UseStandardCXXIncludes && HSOpts.UseStandardSystemIncludes) {
if (HSOpts.UseLibcxx) {
Since you added `HSOpts.UseSt
Author: mramalho
Date: Thu Jun 28 15:08:44 2018
New Revision: 335932
URL: http://llvm.org/viewvc/llvm-project?rev=335932&view=rev
Log:
[analyzer] fix test case expected warning
After r335814, the constraint manager is no longer generating a false bug report
about the division by zero in the test
ldionne added inline comments.
Comment at: include/string:856
+_LIBCPP_METHOD_TEMPLATE_IMPLICIT_INSTANTIATION_VIS
+explicit basic_string(const _Tp& __t, const allocator_type& __a);
+
tcanens wrote:
> ldionne wrote:
> > I think this `explicit` shou
ldionne added a comment.
I guess I'm echoing some of Eric's comments, but does it make sense to build
libc++abi against a system-installed libc++? If not, this change makes sense.
Repository:
rCXXA libc++abi
https://reviews.llvm.org/D48694
___
c
arphaman added inline comments.
Comment at: lib/Frontend/InitHeaderSearch.cpp:474
+ if (Lang.CPlusPlus && !Lang.AsmPreprocessor &&
+ HSOpts.UseStandardCXXIncludes && HSOpts.UseStandardSystemIncludes) {
if (HSOpts.UseLibcxx) {
bruno wrote:
> Since you a
bruno accepted this revision.
bruno added a comment.
This revision is now accepted and ready to land.
LGTM
Comment at: lib/Frontend/InitHeaderSearch.cpp:474
+ if (Lang.CPlusPlus && !Lang.AsmPreprocessor &&
+ HSOpts.UseStandardCXXIncludes && HSOpts.UseStandardSystemInclude
Author: dblaikie
Date: Thu Jun 28 15:58:04 2018
New Revision: 335938
URL: http://llvm.org/viewvc/llvm-project?rev=335938&view=rev
Log:
DebugInfo: Add -gno-gnu-pubnames to allow disabling gnu-pubnames later in the
command line
Modified:
cfe/trunk/include/clang/Driver/Options.td
cfe/trunk/
apazos added inline comments.
Comment at: lib/Sema/SemaDeclAttr.cpp:5301
+
+ if (!isFunctionOrMethod(D)) {
+S.Diag(D->getLocation(), diag::warn_attribute_wrong_decl_type)
aaron.ballman wrote:
> apazos wrote:
> > aaron.ballman wrote:
> > > I don't think you n
apazos updated this revision to Diff 153419.
apazos added a comment.
Updated tests and removed Subjects from Attr.td
https://reviews.llvm.org/D48412
Files:
include/clang/Basic/Attr.td
include/clang/Basic/AttrDocs.td
include/clang/Basic/DiagnosticSemaKinds.td
lib/CodeGen/TargetInfo.cpp
This revision was automatically updated to reflect the committed changes.
Closed by commit rC335940: [frontend] Don't include the C++ stdlib for -x
assembler-with-cpp (authored by arphaman, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D48736?vs=153368&id=153420#toc
Reposit
Author: arphaman
Date: Thu Jun 28 16:23:45 2018
New Revision: 335940
URL: http://llvm.org/viewvc/llvm-project?rev=335940&view=rev
Log:
[frontend] Don't include the C++ stdlib for -x assembler-with-cpp
The new C++ stdlib warning added in r335081 gets
triggered when compiling an assembly file with
deepak2427 added a subscriber: shenhan.
deepak2427 added a comment.
I had based it on the other tests in clang/test/CodeGen.
Do we not need the `-o` to output to standard output?
Or did you mean something else?
Repository:
rC Clang
https://reviews.llvm.org/D48721
__
I had based it on the other tests in clang/test/CodeGen.
Do we not need the `-o` to output to standard output?
Or did you mean something else?
On Thu, Jun 28, 2018 at 10:25 PM Roman Lebedev via Phabricator <
revi...@reviews.llvm.org> wrote:
> lebedev.ri added a comment.
>
> I'm not sure we can us
rsmith added a comment.
This is not specific to the `ASTImporter`; any change to the AST after a call
to `getParents` would have similar problems. Generally, responsibility for
dealing with this must lie with the consumer of the parent map, not with the
`ASTContext`, since the `ASTContext` gene
leonardchan added inline comments.
Comment at: include/clang/Basic/FixedPoint.h:23
+
+class FixedPointNumber {
+ public:
rjmccall wrote:
> ebevhan wrote:
> > rjmccall wrote:
> > > The established naming convention here — as seen in `APInt`, `APFloat`,
> > > `APV
leonardchan updated this revision to Diff 153426.
leonardchan marked 8 inline comments as done.
leonardchan added a comment.
Herald added a subscriber: mgorny.
- Renamed to APFixedPoint
- Added `FixedPointSemantics` to represent saturation and whether or not
padding is involved. Similar to `APFlo
vsapsai created this revision.
vsapsai added reviewers: mclow.lists, EricWF, erik.pilkington.
Herald added a subscriber: dexonsmith.
Can use `decltype` in C++03 because `include/__config` defines a macro when
`decltype` is not available.
https://reviews.llvm.org/D48753
Files:
libcxx/include/m
vsapsai updated this revision to Diff 153430.
vsapsai added a comment.
Herald added a subscriber: dexonsmith.
- Don't check `!__has_construct` for `__construct_range_forward`.
Incompatible types will cause a lack of `construct` but it doesn't mean we
should use memcpy instead. And missing `_Alloc
vsapsai updated this revision to Diff 153431.
vsapsai added a comment.
Try to exclude changes available in parent review from this review.
https://reviews.llvm.org/D48342
Files:
libcxx/include/memory
libcxx/test/std/containers/sequences/vector/vector.cons/construct_iter_iter_alloc.pass.cpp
vsapsai added inline comments.
Comment at: libcxx/include/memory:1479
+struct __has_construct_missing
+: false_type
+{
vsapsai wrote:
> erik.pilkington wrote:
> > vsapsai wrote:
> > > erik.pilkington wrote:
> > > > Shouldn't this be true_type?
> > > I see thi
mclow.lists marked 5 inline comments as done.
mclow.lists added inline comments.
Comment at: include/string:836
_LIBCPP_METHOD_TEMPLATE_IMPLICIT_INSTANTIATION_VIS
-basic_string(const _Tp& __t, size_type __pos, size_type __n,
- const allocator_
Quuxplusone added a comment.
I'm pretty sure that the C++03 standard doesn't permit the implementation to
call any `construct` method here, even if it wanted to. And this adds a couple
hundred LOC to get that (non-standard) behavior. Is there a specific use-case
that you're trying to enable by
vsapsai added a comment.
In https://reviews.llvm.org/D48753#1147500, @Quuxplusone wrote:
> I'm pretty sure that the C++03 standard doesn't permit the implementation to
> call any `construct` method here, even if it wanted to. And this adds a
> couple hundred LOC to get that (non-standard) behav
mclow.lists updated this revision to Diff 153432.
mclow.lists marked 3 inline comments as done.
mclow.lists added a comment.
Update in response to comments.
https://reviews.llvm.org/D48616
Files:
include/memory
include/string
test/std/strings/basic.string/string.cons/copy_assignment.pass.
101 - 200 of 202 matches
Mail list logo
