[PATCH] D57102: [extra] unit tests enable crash-recovery cases on FreeBSD

steveire accepted this revision. steveire added a comment. This revision is now accepted and ready to land. Is this covered by CI? Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57102/new/ https://reviews.llvm.org/D57102 __

[PATCH] D56946: [Documentation] Use HTTPS whenever possible in Clang

This revision was automatically updated to reflect the committed changes. Closed by commit rL351976: [Documentation] Use HTTPS whenever possible (authored by eugenezelenko, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D56946?vs=18264

r351976 - [Documentation] Use HTTPS whenever possible

Author: eugenezelenko Date: Wed Jan 23 12:39:07 2019 New Revision: 351976 URL: http://llvm.org/viewvc/llvm-project?rev=351976&view=rev Log: [Documentation] Use HTTPS whenever possible Differential revision: https://reviews.llvm.org/D56946 Modified: cfe/trunk/docs/AutomaticReferenceCounting.r

[PATCH] D57111: Make Clang not crash on calls to destructors on incomplete pointer types

riccibruno accepted this revision. riccibruno added a comment. This revision is now accepted and ready to land. In D57111#1368277 , @rzhikharevich wrote: > In D57111#1368256 , @riccibruno > wrote: > > > I guess yo

[PATCH] D57113: [clang-tidy] openmp-use-default-none - a new module and a check

Eugene.Zelenko added a comment. Please mention new module in docs/clang-tidy/index.rst. Comment at: docs/ReleaseNotes.rst:76 +- New OpenMP module. + Please put new module before list of new checks. Comment at: docs/ReleaseNotes.rst:83 + +

[PATCH] D57104: [AST] Pack GenericSelectionExpr

riccibruno added a comment. In D57104#1368292 , @steveire wrote: > In D57104#1368072 , @riccibruno > wrote: > > > In D57104#1368055 , @steveire > > wrote: > > > > > Splitti

r351977 - [Documentation] Fix problem in docs/SafeStack.rst introduced in r351976.

Author: eugenezelenko Date: Wed Jan 23 12:51:06 2019 New Revision: 351977 URL: http://llvm.org/viewvc/llvm-project?rev=351977&view=rev Log: [Documentation] Fix problem in docs/SafeStack.rst introduced in r351976. Modified: cfe/trunk/docs/SafeStack.rst Modified: cfe/trunk/docs/SafeStack.rst U

[PATCH] D57113: [clang-tidy] openmp-use-default-none - a new module and a check

lebedev.ri updated this revision to Diff 183160. lebedev.ri marked 3 inline comments as done. lebedev.ri added a comment. Fix documentation nits. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57113/new/ https://reviews.llvm.org/D57113 Files: clan

[PATCH] D57102: [extra] unit tests enable crash-recovery cases on FreeBSD

devnexen added a comment. In D57102#1368309 , @steveire wrote: > Is this covered by CI? The tests pass. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57102/new/ https://reviews.llvm.org/D57102 _

[PATCH] D57102: [extra] unit tests enable crash-recovery cases on FreeBSD

steveire added a comment. The question was whether there is a build-bot for FreeBSD (if there is, then it is covered by CI). Doesn't matter though. If no such bot exists, this isn't the commit to enforce requiring one! Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://

[PATCH] D57098: [WIP][AST] NFC: Introduce new class GenericSelectionExpr::Association

riccibruno abandoned this revision. riccibruno marked 2 inline comments as done. riccibruno added a comment. Closed in favor of D57104 and D57106 . Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57

[PATCH] D57102: [extra] unit tests enable crash-recovery cases on FreeBSD

devnexen added a comment. In D57102#1368373 , @steveire wrote: > The question was whether there is a build-bot for FreeBSD (if there is, then > it is covered by CI). > > Doesn't matter though. If no such bot exists, this isn't the commit to > enforce req

[PATCH] D57102: [extra] unit tests enable crash-recovery cases on FreeBSD

devnexen added a comment. Seems I can t commit myself to the svn repo ... had moved to git ? Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57102/new/ https://reviews.llvm.org/D57102 ___ cfe-commits mail

Re: r347205 - [FileManager] getFile(open=true) after getFile(open=false) should open the file.

For completeness, in case the construction isn't clear, here's a full repro of the symptom. $ cat pch.cc $ cat pch.h #include "foo.h" $ rm bar/foo.h $ for i in {1..300}; do echo '#include "foo'$i'.h"' >> bar/foo.h; done $ for i in {1..300}; do touch bar/foo$i.h; done $ cat client.cc #include "bar/

[PATCH] D57076: [ObjC generics] Fix applying `__kindof` to the type parameter.

jordan_rose added a comment. I //think// this is reasonable but Doug was the one who worked on this. I wonder if it also helps with the test cases in rdar://problem/24619481. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57076/new/ https://reviews.llvm.org/D57076 ___

[PATCH] D57062: [analyzer] Re-enable the "System is over constrained" assertion on optimized builds.

a_sidorin accepted this revision. a_sidorin added a comment. Oh, I remember how much pain have debug mode-only assertions caused. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57062/new/ https://reviews.llvm.org/D57062 _

[PATCH] D57062: [analyzer] Re-enable the "System is over constrained" assertion on optimized builds.

george.karpenkov requested changes to this revision. george.karpenkov added a comment. This revision now requires changes to proceed. We would start getting crashes from `TrustNonnullChecker` if we enable it. Shouldn't those be fixed first? An input which crashes with this assertion is `test/Ana

[PATCH] D50563: Fixed frontend clang tests in windows read-only container

justice_adams updated this revision to Diff 183170. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D50563/new/ https://reviews.llvm.org/D50563 Files: test/Frontend/output-failures.c test/Frontend/stats-file.c Index: test/Frontend/stats-file.c =

[PATCH] D57062: [analyzer] Re-enable the "System is over constrained" assertion on optimized builds.

NoQ added a subscriber: alexfh. NoQ added a comment. I think there are much more problems that we will find this way, starting from long foo(long x) { unsigned y = (unsigned)x; if (y <= -1) return 0; return (y + 2) < 2; // crash :( } I don't mind delaying this patch unti

[PATCH] D57102: [extra] unit tests enable crash-recovery cases on FreeBSD

steveire added a comment. You may need to accept the new license terms: http://lists.llvm.org/pipermail/llvm-dev/2019-January/129069.html Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57102/new/ https://reviews.llvm.org/D57102 __

[PATCH] D50563: Fixed frontend clang tests in windows read-only container

justice_adams added a comment. @stella.stamenova Good suggestion, I think you are right. I have updated the diff to use %t, that way any end-users can control the way their lit test work in a read-only mount by simply altering the test_exec_root in their config CHANGES SINCE LAST ACTION http

[PATCH] D57102: [extra] unit tests enable crash-recovery cases on FreeBSD

devnexen added a comment. I did filled the form already ... do not know if it needs days to take effects ... Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57102/new/ https://reviews.llvm.org/D57102 ___

[PATCH] D50563: Fixed frontend clang tests in windows read-only container

stella.stamenova requested changes to this revision. stella.stamenova added a comment. This revision now requires changes to proceed. Thanks. You have to change a couple of the locations to '%T' though - '%t' is a file and '%T' is a directory - and you're using '%t' as a directory in a couple of

[PATCH] D56803: clang -dumpversion returns 4.2.1 for legacy reason, update it

sylvestre.ledru added a comment. @ddunbar is that ok with you? Thanks See https://bugs.llvm.org/show_bug.cgi?id=38836 https://bugs.launchpad.net/ubuntu/+source/llvm-defaults/+bug/1810860 Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56803/new/ https://reviews.llv

[PATCH] D57114: Remove Expr sugar decorating the CXXUuidofExpr node.

void marked 2 inline comments as done. void added inline comments. Comment at: lib/Sema/SemaTemplate.cpp:6311 if (isa(E)) { - Converted = TemplateArgument(ArgResult.get()); + Converted = TemplateArgument(ArgResult.get()->IgnoreImpCasts()); br

[PATCH] D57114: Remove Expr sugar decorating the CXXUuidofExpr node.

void updated this revision to Diff 183180. void added a comment. Move test to proper place and add "-verify" flag. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57114/new/ https://reviews.llvm.org/D57114 Files: lib/Sema/SemaTemplate.cpp test/SemaCXX/PR40395.

[PATCH] D28248: Work around GCC PR37804

mclow.lists closed this revision. mclow.lists added a comment. I changed the `_LIBCPP_TYPE_VIS_ONLY` to `_LIBCPP_TEMPLATE_VIS` and committed this as revision 351993. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D28248/new/ https://reviews.llvm.org/D28248

[PATCH] D26110: Add a check for GCC to the _LIBCPP_EXPLICIT define

mclow.lists closed this revision. mclow.lists added a comment. Yes, this appears to have been landed. Closing. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D26110/new/ https://reviews.llvm.org/D26110 ___ cfe-commits mailing list cfe-commits

[PATCH] D57114: Remove Expr sugar decorating the CXXUuidofExpr node.

riccibruno added inline comments. Comment at: lib/Sema/SemaTemplate.cpp:6311 if (isa(E)) { - Converted = TemplateArgument(ArgResult.get()); + Converted = TemplateArgument(ArgResult.get()->IgnoreImpCasts()); break; A remark wh

[PATCH] D50563: Fixed frontend clang tests in windows read-only container

justice_adams updated this revision to Diff 183197. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D50563/new/ https://reviews.llvm.org/D50563 Files: test/Frontend/output-failures.c test/Frontend/stats-file.c Index: test/Frontend/stats-file.c =

[PATCH] D50563: Fixed frontend clang tests in windows read-only container

justice_adams added a comment. @stella.stamenova Thanks for the input, what about now? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D50563/new/ https://reviews.llvm.org/D50563 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://

[PATCH] D54399: Move ExprMutationAnalyzer to Tooling/Analysis (1/3)

shuaiwang added a comment. Friendly ping @EricWF Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54399/new/ https://reviews.llvm.org/D54399 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org

r351997 - [Sema] Fix Modified Type in address_space AttributedType

Author: leonardchan Date: Wed Jan 23 16:11:35 2019 New Revision: 351997 URL: http://llvm.org/viewvc/llvm-project?rev=351997&view=rev Log: [Sema] Fix Modified Type in address_space AttributedType This is a fix for https://reviews.llvm.org/D51229 where we pass the address_space qualified type as th

[PATCH] D55447: [Sema] Fix Modified Type in address_space AttributedType

This revision was automatically updated to reflect the committed changes. Closed by commit rL351997: [Sema] Fix Modified Type in address_space AttributedType (authored by leonardchan, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D554

[libclc] r352000 - Creating release candidate rc1 from release_800 branch

Author: hans Date: Wed Jan 23 16:44:29 2019 New Revision: 352000 URL: http://llvm.org/viewvc/llvm-project?rev=352000&view=rev Log: Creating release candidate rc1 from release_800 branch Added: libclc/tags/RELEASE_800/ libclc/tags/RELEASE_800/rc1/ - copied from r351999, libclc/branch

[libunwind] r352000 - Creating release candidate rc1 from release_800 branch

Author: hans Date: Wed Jan 23 16:44:29 2019 New Revision: 352000 URL: http://llvm.org/viewvc/llvm-project?rev=352000&view=rev Log: Creating release candidate rc1 from release_800 branch Added: libunwind/tags/RELEASE_800/ libunwind/tags/RELEASE_800/rc1/ - copied from r351999, libunwi

[PATCH] D57076: [ObjC generics] Fix applying `__kindof` to the type parameter.

vsapsai added a comment. In D57076#1368407 , @jordan_rose wrote: > I //think// this is reasonable but Doug was the one who worked on this. I > wonder if it also helps with the test cases in rdar://problem/24619481. Yep, it helps with the test cases in r

[PATCH] D56927: Disable PIC/PIE for MSP430 target

dmikushin updated this revision to Diff 183214. dmikushin added a comment. Adding test case Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56927/new/ https://reviews.llvm.org/D56927 Files: lib/Driver/ToolChains/MSP430.h test/CodeGen/msp430-reloc.c Index: te

[PATCH] D57076: [ObjC generics] Fix applying `__kindof` to the type parameter.

vsapsai updated this revision to Diff 183216. vsapsai added a comment. Update diff to ignore parent patch. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57076/new/ https://reviews.llvm.org/D57076 Files: clang/lib/AST/Type.cpp clang/test/SemaObjC/kindof.m Index: clang/test/SemaObjC

[PATCH] D57076: [ObjC generics] Fix applying `__kindof` to the type parameter.

vsapsai updated this revision to Diff 183215. vsapsai added a comment. - Add a test that `__kindof` works with type sugar such as typedef. - Use early return to reduce nesting. Also it's consistent with the previous early returns. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57076/new/

r352003 - [Sanitizers] UBSan unreachable incompatible with ASan in the presence of `noreturn` calls

Author: yln Date: Wed Jan 23 17:06:19 2019 New Revision: 352003 URL: http://llvm.org/viewvc/llvm-project?rev=352003&view=rev Log: [Sanitizers] UBSan unreachable incompatible with ASan in the presence of `noreturn` calls Summary: UBSan wants to detect when unreachable code is actually reached, so

[PATCH] D16541: [libc++] Renable test/std/re/re.alg/re.alg.match/awk.pass.cpp

mclow.lists closed this revision. mclow.lists added a comment. Herald added a reviewer: EricWF. For some reason, this test was not marked `// XFAIL gnu-linux` like all the other tests that use the `LOCALE_cs_CZ_ISO8859_2` locale. In revision 352006, I uncommented this entire test, and added the X

[PATCH] D55850: [OpenCL] Allow address spaces as method qualifiers

Anastasia added a comment. In D55850#1366709 , @rjmccall wrote: > Most of the remaining OpenCL checks are for OpenCL-specific logic like > inferring the default address space, and yeah, we could probably make that a > target option or something. We hav

r352011 - Improve diagnostic for -fvisibility mismatch between module/PCH build

Author: rsmith Date: Wed Jan 23 18:41:46 2019 New Revision: 352011 URL: http://llvm.org/viewvc/llvm-project?rev=352011&view=rev Log: Improve diagnostic for -fvisibility mismatch between module/PCH build and use. Modified: cfe/trunk/include/clang/Basic/LangOptions.def Modified: cfe/trunk/incl

[PATCH] D55500: [Builtins] Implement __builtin_is_constant_evaluated for use in C++2a

rsmith added a comment. Thanks for the added tests! They seem to have found a bug in the handling of local const integral variables. Comment at: test/CodeGenCXX/builtin-is-constant-evaluated.cpp:41-42 + +// CHECK-STATIC-DAG: @p = global i32 26, +CONSTINIT int p = f(); // f().m

[libunwind] r352016 - [libunwind] Don't abort if encoutering invalid .eh_frame_hdr

Author: phosek Date: Wed Jan 23 19:04:42 2019 New Revision: 352016 URL: http://llvm.org/viewvc/llvm-project?rev=352016&view=rev Log: [libunwind] Don't abort if encoutering invalid .eh_frame_hdr Recent Linux kernel release has introduced a bug as part of the ORC rollout where the vDSO has a valid

[PATCH] D57135: [ExprConstant] Unify handling of array init with lvalues.

efriedma created this revision. efriedma added a reviewer: rsmith. This should have no functional effect; evaluation is still generating the same result. But the assertions and comments should make it more clear what's actually happening. Part of a fix to https://bugs.llvm.org/show_bug.cgi?id

[PATCH] D57136: [CMake][Fuchsia] Enable hermetic static libc++abi for Fuchsia

phosek created this revision. phosek added reviewers: mcgrathr, juliehockett, jakehehrlich. Herald added subscribers: cfe-commits, mgorny. Herald added a reviewer: EricWF. Similarly to libc++, we want to use hermetic static libc++abi. Repository: rC Clang https://reviews.llvm.org/D57136 File

r352018 - [CMake][Fuchsia] Enable hermetic static libc++abi for Fuchsia

Author: phosek Date: Wed Jan 23 19:24:37 2019 New Revision: 352018 URL: http://llvm.org/viewvc/llvm-project?rev=352018&view=rev Log: [CMake][Fuchsia] Enable hermetic static libc++abi for Fuchsia Similarly to libc++, we want to use hermetic static libc++abi. Differential Revision: https://reviews

[PATCH] D57136: [CMake][Fuchsia] Enable hermetic static libc++abi for Fuchsia

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 rC352018: [CMake][Fuchsia] Enable hermetic static libc++abi for Fuchsia (authored by phosek, committed by ). Changed prior

[PATCH] D51329: [Attribute/Diagnostics] Print macro instead of whole attribute for address_space

leonardchan updated this revision to Diff 183242. leonardchan added a comment. - Rebase after D55447 Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D51329/new/ https://reviews.llvm.org/D51329 Files: clang/include/clang/AST/ASTCo

[PATCH] D51329: [Attribute/Diagnostics] Print macro instead of whole attribute for address_space

leonardchan marked 2 inline comments as done. leonardchan added inline comments. Comment at: clang/lib/AST/TypePrinter.cpp:958-964 + // Remove the address_space qualifier so it does not get printed. We + // instead want to print the macro only. + SplitQualType Spli

[PATCH] D55500: [Builtins] Implement __builtin_is_constant_evaluated for use in C++2a

EricWF marked 6 inline comments as done. EricWF added inline comments. Comment at: test/CodeGenCXX/builtin-is-constant-evaluated.cpp:41-42 + +// CHECK-STATIC-DAG: @p = global i32 26, +CONSTINIT int p = f(); // f().m == 13; initialized to 26 + rsmith wrote: > Do w

r352025 - Reland r345009 "[DebugInfo] Generate debug information for labels."

Author: hsiangkai Date: Wed Jan 23 21:34:29 2019 New Revision: 352025 URL: http://llvm.org/viewvc/llvm-project?rev=352025&view=rev Log: Reland r345009 "[DebugInfo] Generate debug information for labels." Generate DILabel metadata and call llvm.dbg.label after label statement to associate the meta

[PATCH] D45045: [DebugInfo] Generate debug information for labels.

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 rL352025: Reland r345009 "[DebugInfo] Generate debug information for labels." (authored by HsiangKai, committed by ). Chang

[PATCH] D57076: [ObjC generics] Fix applying `__kindof` to the type parameter.

doug.gregor accepted this revision. doug.gregor added a comment. This revision is now accepted and ready to land. One minor request about regarding a dyn_cast, but this looks like the right approach. Thank you! Comment at: clang/lib/AST/Type.cpp:1293 + +const auto *newAttr

[PATCH] D55500: [Builtins] Implement __builtin_is_constant_evaluated for use in C++2a

EricWF updated this revision to Diff 183248. EricWF marked 2 inline comments as done. EricWF added a comment. - Add or correct tests as requested. - Fix code gen for initializers of reference types to correctly evaluate the initializer as a constant expression. CHANGES SINCE LAST ACTION https

[PATCH] D57072: Don't codegen an unreachable return block

rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. Sure, I have no objection to doing this at this point. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57072/new/ https://reviews.llvm.org/D57072

[PATCH] D55850: [OpenCL] Allow address spaces as method qualifiers

rjmccall added a comment. In D55850#1368767 , @Anastasia wrote: > In D55850#1366709 , @rjmccall wrote: > > > Most of the remaining OpenCL checks are for OpenCL-specific logic like > > inferring the default address

[PATCH] D56936: Fix handling of overriden methods during ASTImport

shafik marked 17 inline comments as done. shafik added a comment. @martong I don't follow the discussion on `VisitParmVarDecl` an what seems like an alternate fix. Can you elaborate? Comment at: lib/AST/ASTImporter.cpp:3042 + if (FoundByLookup) { +if (auto *MD = dyn_cas

[PATCH] D56936: Fix handling of overriden methods during ASTImport

shafik updated this revision to Diff 183251. shafik added a comment. Addressing comments on naming in the unit test and refactoring of duplicated code. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56936/new/ https://reviews.llvm.org/D56936 Files: lib/AST/ASTImporter.cpp unittests

[PATCH] D56318: [HIP] Fix size_t for MSVC environment

rjmccall added inline comments. Comment at: include/clang/Basic/TargetInfo.h:52 -/// Exposes information about the current target. -/// -class TargetInfo : public RefCountedBase { - std::shared_ptr TargetOpts; - llvm::Triple Triple; -protected: - // Target values set by the

[PATCH] D57072: Don't codegen an unreachable return block

bmoody added a comment. Great, I don't have commit access so you're welcome to commit on my behalf. Thanks! Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57072/new/ https://reviews.llvm.org/D57072 ___ cfe-commits m

[PATCH] D56909: [clang-format] Fix line parsing for noexcept lambdas

hultman added a comment. @benhamilton Would you mind landing the patch? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56909/new/ https://reviews.llvm.org/D56909 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang-tools-extra] r352031 - [extra] unit tests enable crash-recovery cases on FreeBSD

Author: devnexen Date: Wed Jan 23 23:58:42 2019 New Revision: 352031 URL: http://llvm.org/viewvc/llvm-project?rev=352031&view=rev Log: [extra] unit tests enable crash-recovery cases on FreeBSD Seems the previous statement does not hold up anymore. Reviewers: steveire Reviewed By: steveire Diff

[PATCH] D57102: [extra] unit tests enable crash-recovery cases on FreeBSD

This revision was automatically updated to reflect the committed changes. Closed by commit rCTE352031: [extra] unit tests enable crash-recovery cases on FreeBSD (authored by devnexen, committed by ). Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57102

<    1   2