[PATCH] D15836: [libcxx] Fix undefined behavior in forward_list

EricWF created this revision. EricWF added a reviewer: mclow.lists. EricWF added a subscriber: cfe-commits. This patch is similar to the fix but it has a few differences. This patch doesn't use a `__link_pointer` typedef because we don't need to change the linked list pointers because `forward_

Re: [PATCH] D15836: [libcxx] Fix undefined behavior in forward_list

EricWF updated this revision to Diff 43824. EricWF added a comment. forward_list and it's iterators should allow incomplete types in C++17 and beyond. I updated the patch to fix this issue as a drive-by. http://reviews.llvm.org/D15836 Files: include/__config include/forward_list test/std

Re: [PATCH] D15709: [X86] Support 'interrupt' attribute for x86

ABataev added a comment. Aaron, thanks for the review! Comment at: include/clang/Basic/DiagnosticSemaKinds.td:2504 @@ -2493,3 +2503,3 @@ // Availability attribute def warn_availability_unknown_platform : Warning< aaron.ballman wrote: > >> It would be good to

Re: [PATCH] D15710: [clang-tidy] Add non-inline function definition and variable definition check in header files.

hokein updated this revision to Diff 43826. hokein added a comment. Move unittest to lit test. http://reviews.llvm.org/D15710 Files: clang-tidy/misc/CMakeLists.txt clang-tidy/misc/DefinitionsInHeadersCheck.cpp clang-tidy/misc/DefinitionsInHeadersCheck.h clang-tidy/misc/MiscTidyModule.cp

Re: [PATCH] D15709: [X86] Support 'interrupt' attribute for x86

ABataev updated this revision to Diff 43828. ABataev added a comment. Fix after review. After some discussion with gcc guys it was decided to allow to use static member functions as interrupt handler. http://reviews.llvm.org/D15709 Files: include/clang/Basic/Attr.td include/clang/Basic/Att

Re: [PATCH] D15710: [clang-tidy] Add non-inline function definition and variable definition check in header files.

hokein marked 5 inline comments as done. hokein added a comment. http://reviews.llvm.org/D15710 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D15805: [clang-tidy] Cleanup code in CERT module.

hokein added a comment. So shoud we need to rename to `cert` at this patch? Repository: rL LLVM http://reviews.llvm.org/D15805 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D15639: [clang-format] Ensure Sort include is stable with negative Priority

jeanphilippeD abandoned this revision. jeanphilippeD added a comment. This was fixed by other patch. http://reviews.llvm.org/D15639 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D14727: [Driver] Adapt Linux::GCCVersion::Parse to match GCC 5 installations

thiagomacieira added a comment. Ping? http://reviews.llvm.org/D14727 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D14727: [Driver] Adapt Linux::GCCVersion::Parse to match GCC 5 installations

thiagomacieira added a comment. Ah, I see a comment. Sorry about that. I'll see if I can fix it, but I had no failures when I tried... http://reviews.llvm.org/D14727 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-

Re: [PATCH] D15266: [clang-format] Handle \n the same way as std::endl with stream operator.

jeanphilippeD added a comment. Ping. Is that patch of interest? http://reviews.llvm.org/D15266 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r256672 - [X86][PKU] add clang intrinsic for {RD|WR}PKRU

Author: abadouh Date: Thu Dec 31 08:14:07 2015 New Revision: 256672 URL: http://llvm.org/viewvc/llvm-project?rev=256672&view=rev Log: [X86][PKU] add clang intrinsic for {RD|WR}PKRU Differential Revision: http://reviews.llvm.org/D15837 Added: cfe/trunk/lib/Headers/pkuintrin.h cfe/trunk/te

Re: [PATCH] D15823: Support virtual-near-miss check.

congliu marked 2 inline comments as done. Comment at: clang-tidy/misc/VirtualNearMissCheck.cpp:79 @@ +78,3 @@ + if (BaseReturnType->isReferenceType() && DerivedReturnType->isReferenceType()){ +BaseReturnType = BaseReturnType.getNonReferenceType(); +DerivedReturnType = De

Re: [PATCH] D12834: add gcc abi_tag support

elizafox added a comment. In http://reviews.llvm.org/D12834#316988, @foutrelis wrote: > In http://reviews.llvm.org/D12834#314136, @elizafox wrote: > > > In http://reviews.llvm.org/D12834#311890, @foutrelis wrote: > > > > > We have received a few reports of clang crashes after applying the > > >

Re: [PATCH] D15805: [clang-tidy] Cleanup code in CERT module.

aaron.ballman added a comment. In http://reviews.llvm.org/D15805#318307, @hokein wrote: > So shoud we need to rename to `cert` at this patch? Yes, but just the namespaces as you do in this patch (not anything in user documentation, for instance). This patch LG; if you need me to commit on your

Re: [PATCH] D15709: [X86] Support 'interrupt' attribute for x86

aaron.ballman added inline comments. Comment at: include/clang/Basic/DiagnosticSemaKinds.td:259 @@ +258,3 @@ +def err_anyx86_interrupt_attribute : Error< + "interrupt service routine %select{must have void return value|" + "can only have a pointer argument and an optional intege

Re: [libcxx] r249798 - Split out of .

On Wed, Dec 30, 2015 at 8:28 PM, Richard Smith wrote: > On Wed, Dec 30, 2015 at 1:17 PM, Nico Weber wrote: > >> One problem with this patch: stdio.h can be used in .c files, and when >> building .c files with -gnu99 -pedantic, >> > > Do you mean -std=gnu89? > Sorry, I meant -std=gnu99 -pedantic

[PATCH] D15839: Move _xgetbv to immintrin.h

ehsan created this revision. ehsan added a reviewer: hansw. ehsan added a subscriber: cfe-commits. The documentation in https://msdn.microsoft.com/en-us/library/hh977022.aspx says that this intrinsic is defined in immintrin.h, and that's the header where MSVC declares this intrinsic, but clang-cl

Re: [PATCH] D15839: Move _xgetbv to immintrin.h

ehsan abandoned this revision. ehsan added a comment. Oops, sorry, I submitted this by mistake. http://reviews.llvm.org/D15839 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [libcxx] r249798 - Split out of .

On Thu, Dec 31, 2015 at 10:52 AM, Nico Weber wrote: > On Wed, Dec 30, 2015 at 8:28 PM, Richard Smith > wrote: > >> On Wed, Dec 30, 2015 at 1:17 PM, Nico Weber wrote: >> >>> One problem with this patch: stdio.h can be used in .c files, and when >>> building .c files with -gnu99 -pedantic, >>> >>

r256683 - [TrailingObjects] Convert classes in OpenMPClause.h

Author: jyknight Date: Thu Dec 31 18:38:24 2015 New Revision: 256683 URL: http://llvm.org/viewvc/llvm-project?rev=256683&view=rev Log: [TrailingObjects] Convert classes in OpenMPClause.h Modified: cfe/trunk/include/clang/AST/OpenMPClause.h cfe/trunk/lib/AST/OpenMPClause.cpp Modified: cfe

r256684 - [TrailingObjects] Fix "Convert classes in OpenMPClause.h" for MSVC

Author: jyknight Date: Thu Dec 31 19:12:48 2015 New Revision: 256684 URL: http://llvm.org/viewvc/llvm-project?rev=256684&view=rev Log: [TrailingObjects] Fix "Convert classes in OpenMPClause.h" for MSVC "friend class OMPVarListClause" -> "friend OMPVarListClause". It's a template, not a class. M

r256686 - Reimplement __readeflags and __writeeflags on top of intrinsics

Author: majnemer Date: Fri Jan 1 00:50:08 2016 New Revision: 256686 URL: http://llvm.org/viewvc/llvm-project?rev=256686&view=rev Log: Reimplement __readeflags and __writeeflags on top of intrinsics Lean on LLVM to provide this functionality now that it provides the necessary intrinsics. Modifie