r319881 - [libclang] Add function to get the buffer for a file

2017-12-06 Thread Erik Verbruggen via cfe-commits
Author: erikjv Date: Wed Dec 6 01:02:52 2017 New Revision: 319881 URL: http://llvm.org/viewvc/llvm-project?rev=319881&view=rev Log: [libclang] Add function to get the buffer for a file This can be used by clients in conjunction with an offset returned by e.g. clang_getFileLocation. Now those cli

[PATCH] D40643: [libclang] Add function to get the buffer for a file

2017-12-06 Thread Erik Verbruggen via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL319881: [libclang] Add function to get the buffer for a file (authored by erikjv). Changed prior to commit: https://reviews.llvm.org/D40643?vs=124904&id=125677#toc Repository: rL LLVM https://review

[PATCH] D40884: [Index] Add setPreprocessor member to IndexDataConsumer.

2017-12-06 Thread Eric Liu via Phabricator via cfe-commits
ioeric created this revision. This enables us to use information in Preprocessor when handling symbol occurrences. Repository: rC Clang https://reviews.llvm.org/D40884 Files: include/clang/Index/IndexDataConsumer.h include/clang/Index/IndexingAction.h lib/Index/IndexingAction.cpp too

[PATCH] D40884: [Index] Add setPreprocessor member to IndexDataConsumer.

2017-12-06 Thread Haojian Wu via Phabricator via cfe-commits
hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land. LGTM. Comment at: include/clang/Index/IndexingAction.h:14 #include "clang/Basic/LLVM.h" +#include "clang/Lex/Preprocessor.h" #include "llvm/ADT/ArrayRef.h"

r319883 - [OpenCL] Fix layering violation by getOpenCLTypeAddrSpace

2017-12-06 Thread Sven van Haastregt via cfe-commits
Author: svenvh Date: Wed Dec 6 02:11:28 2017 New Revision: 319883 URL: http://llvm.org/viewvc/llvm-project?rev=319883&view=rev Log: [OpenCL] Fix layering violation by getOpenCLTypeAddrSpace Commit 7ac28eb0a5 / r310911 ("[OpenCL] Allow targets to select address space per type", 2017-08-15) made B

[PATCH] D40838: [OpenCL] Fix layering violation by getOpenCLTypeAddrSpace

2017-12-06 Thread Sven van Haastregt via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL319883: [OpenCL] Fix layering violation by getOpenCLTypeAddrSpace (authored by svenvh). Changed prior to commit: https://reviews.llvm.org/D40838?vs=125554&id=125682#toc Repository: rL LLVM https://r

[PATCH] D40838: [OpenCL] Fix layering violation by getOpenCLTypeAddrSpace

2017-12-06 Thread Sven van Haastregt via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC319883: [OpenCL] Fix layering violation by getOpenCLTypeAddrSpace (authored by svenvh). Repository: rC Clang https://reviews.llvm.org/D40838 Files: include/clang/AST/ASTContext.h include/clang/Bas

[PATCH] D37813: clang-format: better handle namespace macros

2017-12-06 Thread Manuel Klimek via Phabricator via cfe-commits
klimek added a comment. In https://reviews.llvm.org/D37813#945125, @Typz wrote: > I don't think this is really relevant for this tool: if someone changes the > implementation of the macro, then *they* must indeed if it should not be > formatted like a namespace (and keep the clang-format config

[PATCH] D40488: [clangd] Implemented tracing using Context

2017-12-06 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov updated this revision to Diff 125690. ilya-biryukov added a comment. Herald added a subscriber: klimek. - Updated tracing to use the new Context implementation. - Restored global tracer. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D40488 Files: clangd/ClangdUn

[PATCH] D40489: [clangd] Changed tracing interfaces

2017-12-06 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov updated this revision to Diff 125696. ilya-biryukov added a comment. Herald added a subscriber: klimek. - Update the patch to accomodate changes from tracing and Context patches. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D40489 Files: clangd/Trace.cpp clan

[PATCH] D33467: Fix LLVM build errors if necent build of GCC 7 is used

2017-12-06 Thread Guilherme Amadio via Phabricator via cfe-commits
amadio added a comment. The problems fixed here also happen when compiling with Clang-5.0 and C++17 enabled. What happens is that the assignment to `ProfileFileName` needs a conversion of `PGOTestProfileFile` from the `opt` type to `std::string`, but the compiler is trying to use a deleted cons

[PATCH] D40888: [ARM] ACLE parallel arithmetic and DSP style multiplications

2017-12-06 Thread Sjoerd Meijer via Phabricator via cfe-commits
SjoerdMeijer created this revision. Herald added subscribers: kristof.beyls, javed.absar, aemerson. This is a follow up of r302131, in which we forgot to add SemaChecking tests. Adding these tests revealed two problems which have been fixed: - added missing intrinsic __qdbl, - properly ran

[PATCH] D40485: [clangd] Introduced a Context that stores implicit data

2017-12-06 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov updated this revision to Diff 125712. ilya-biryukov marked an inline comment as done. ilya-biryukov added a comment. - Removed unused helpers from Context.h - Use assert instead of llvm_unreachable in getExisiting(Key<>) Repository: rCTE Clang Tools Extra https://reviews.llvm.or

[PATCH] D20124: [PCH] Serialize skipped preprocessor ranges

2017-12-06 Thread Nikolai Kosjar via Phabricator via cfe-commits
nik added a comment. In https://reviews.llvm.org/D20124#943592, @cameron314 wrote: > Here's the final patch that fixes `clang_getSkippedRegions` with regions in > the preamble (as well as serializing the skipped regions in the PCH file). Works fine for me, thanks! Test from the bug report is f

[PATCH] D40488: [clangd] Implemented tracing using Context

2017-12-06 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov updated this revision to Diff 125713. ilya-biryukov added a comment. - Use getExistingKey instead of getPtr in JSONRPCDispatcher. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D40488 Files: clangd/ClangdUnit.cpp clangd/JSONRPCDispatcher.cpp clangd/Trace.cpp

[PATCH] D40888: [ARM] ACLE parallel arithmetic and DSP style multiplications

2017-12-06 Thread Sam Parker via Phabricator via cfe-commits
samparker added a comment. Thanks for looking into this! Comment at: include/clang/Basic/BuiltinsARM.def:39 BUILTIN(__builtin_arm_qsub, "iii", "nc") +BUILTIN(__builtin_arm_qdbl, "ii", "nc") BUILTIN(__builtin_arm_ssat, "iiUi", "nc") Do we now need a codegen te

[PATCH] D40895: Ignore pointers to incomplete types when diagnosing misaligned addresses

2017-12-06 Thread Roger Ferrer Ibanez via Phabricator via cfe-commits
rogfer01 created this revision. This is a fix for PR35509 in which we crash because we attempt to compute the alignment of an incomplete type. https://reviews.llvm.org/D40895 Files: lib/Sema/SemaChecking.cpp test/SemaCXX/address-packed.cpp Index: test/SemaCXX/address-packed.cpp =

[PATCH] D40888: [ARM] ACLE parallel arithmetic and DSP style multiplications

2017-12-06 Thread Sjoerd Meijer via Phabricator via cfe-commits
SjoerdMeijer added inline comments. Comment at: include/clang/Basic/BuiltinsARM.def:39 BUILTIN(__builtin_arm_qsub, "iii", "nc") +BUILTIN(__builtin_arm_qdbl, "ii", "nc") BUILTIN(__builtin_arm_ssat, "iiUi", "nc") samparker wrote: > Do we now need a codegen tests

r319896 - [OPENMP] Initial codegen for `teams distribute simd` directive.

2017-12-06 Thread Alexey Bataev via cfe-commits
Author: abataev Date: Wed Dec 6 06:31:09 2017 New Revision: 319896 URL: http://llvm.org/viewvc/llvm-project?rev=319896&view=rev Log: [OPENMP] Initial codegen for `teams distribute simd` directive. Host + default devices codegen for `teams distribute simd` directive. Added: cfe/trunk/test/Op

[PATCH] D40884: [Index] Add setPreprocessor member to IndexDataConsumer.

2017-12-06 Thread Marc-Andre Laperle via Phabricator via cfe-commits
malaperle added a comment. You can get the preprocessor from the ASTContext, no? Repository: rC Clang https://reviews.llvm.org/D40884 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi

[PATCH] D39457: [OPENMP] Current status of OpenMP support.

2017-12-06 Thread Alexey Bataev via Phabricator via cfe-commits
ABataev updated this revision to Diff 125722. ABataev added a comment. Update Repository: rC Clang https://reviews.llvm.org/D39457 Files: docs/OpenMPSupport.rst docs/index.rst Index: docs/index.rst === --- docs/index.rst +

[PATCH] D40864: [Darwin] Add a new -mstack-probe option and enable by default

2017-12-06 Thread Amara Emerson via Phabricator via cfe-commits
aemerson added inline comments. Comment at: lib/CodeGen/BackendUtil.cpp:442 Options.DebuggerTuning = CodeGenOpts.getDebuggerTuning(); + Options.EnableStackProbe = CodeGenOpts.StackProbe; ahatanak wrote: > Is there a reason you can't use function attributes

[PATCH] D36672: [clang-tidy] readability-non-const-parameter: fixit on all function declarations

2017-12-06 Thread Anders Rönnholm via Phabricator via cfe-commits
AndersRonnholm abandoned this revision. AndersRonnholm added a comment. Fixed by https://reviews.llvm.org/rL319021. At least for c/c++ not sure if it handles objective-c. Repository: rL LLVM https://reviews.llvm.org/D36672 ___ cfe-commits mailin

r319897 - [OPENMP] Improve error message for mapping union members.

2017-12-06 Thread Alexey Bataev via cfe-commits
Author: abataev Date: Wed Dec 6 07:04:36 2017 New Revision: 319897 URL: http://llvm.org/viewvc/llvm-project?rev=319897&view=rev Log: [OPENMP] Improve error message for mapping union members. Modified: cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td cfe/trunk/test/OpenMP/target_map_m

[PATCH] D40485: [clangd] Introduced a Context that stores implicit data

2017-12-06 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. There's a couple of points where we might be a bit stuck on complexity vs X tradeoffs (map + contextbuilder, and some of the convenience APIs). Just want to mention: if I find these things complex, it doesn't mean I think they're bad code, or even that you should agre

[PATCH] D39027: [docs][refactor] Add a new tutorial that talks about how one can implement refactoring actions

2017-12-06 Thread Manuel Klimek via Phabricator via cfe-commits
klimek added inline comments. Comment at: docs/RefactoringActionTutorial.rst:7 + + This tutorial talks about a work-in-progress library in Clang. + Some of the described features might not be available yet in trunk, but should hokein wrote: > I'm a bit concern

[PATCH] D40485: [clangd] Introduced a Context that stores implicit data

2017-12-06 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. Oops, forgot one important question! BTW, phabricator seems to have misaligned all my previous comments (you updated the diff before I submitted them I guess). Let me know if any of them don't make sense now! Comment at: clangd/Context.h:63 +/// us

[PATCH] D37813: clang-format: better handle namespace macros

2017-12-06 Thread Francois Ferrand via Phabricator via cfe-commits
Typz added a comment. > Ok, that's probably where our different opinions come from - I would want a > macro to be formatted to reflect how it's implemented, because otherwise I'm > going to be surprised when I look at the implementation, and I consider > surprises to be something to avoid in pr

[PATCH] D40897: [clangd] Introduce a "Symbol" class.

2017-12-06 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. Herald added subscribers: mgorny, klimek. - The "Symbol" class represents a C++ symbol in the codebase, containing all the information of a C++ symbol needed by clangd. clangd will use it in clangd's AST/dynamic index and global/static index (code completion and cod

[PATCH] D40485: [clangd] Introduced a Context that stores implicit data

2017-12-06 Thread Eric Liu via Phabricator via cfe-commits
ioeric added inline comments. Comment at: clangd/Context.h:63 +/// used as parents for some other Contexts. +class Context { +public: sammccall wrote: > I think we should strongly consider calling the class Ctx over Context. It's > going to appear in many functi

[PATCH] D30691: [analyzer] Support for naive cross translational unit analysis

2017-12-06 Thread Zoltán Gera via Phabricator via cfe-commits
gerazo added a comment. The code modifications are coming soon (after doing some extensive testing) for the scan-build part. Comment at: tools/scan-build-py/libscanbuild/analyze.py:223 +ctu_config = get_ctu_config(args) +if ctu_config.collect: +shutil.rmtree(ct

[PATCH] D40884: [Index] Add setPreprocessor member to IndexDataConsumer.

2017-12-06 Thread Eric Liu via Phabricator via cfe-commits
ioeric added a comment. In https://reviews.llvm.org/D40884#946506, @malaperle wrote: > You can get the preprocessor from the ASTContext, no? I don't think `ASTContext` contains preprocessor information. Repository: rC Clang https://reviews.llvm.org/D40884 ___

[PATCH] D40884: [Index] Add setPreprocessor member to IndexDataConsumer.

2017-12-06 Thread Marc-Andre Laperle via Phabricator via cfe-commits
malaperle added a comment. In https://reviews.llvm.org/D40884#946630, @ioeric wrote: > In https://reviews.llvm.org/D40884#946506, @malaperle wrote: > > > You can get the preprocessor from the ASTContext, no? > > > I don't think `ASTContext` contains preprocessor information. My bad, it was the

[PATCH] D40888: [ARM] ACLE parallel arithmetic and DSP style multiplications

2017-12-06 Thread Sam Parker via Phabricator via cfe-commits
samparker accepted this revision. samparker added a comment. This revision is now accepted and ready to land. Great, LGTM, many thanks for doing this! https://reviews.llvm.org/D40888 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lis

[PATCH] D40901: Refactor lazy loading of template specializations. NFC

2017-12-06 Thread Vassil Vassilev via Phabricator via cfe-commits
v.g.vassilev created this revision. Unify the loading of lazy template specializations across class, function and variable templates. Repository: rL LLVM https://reviews.llvm.org/D40901 Files: include/clang/AST/DeclTemplate.h lib/AST/DeclTemplate.cpp Index: lib/AST/DeclTemplate.cpp =

[PATCH] D40903: [Sanitizers] Basic Solaris sanitizer support (PR 33274)

2017-12-06 Thread Rainer Orth via Phabricator via cfe-commits
ro created this revision. ro added a project: Sanitizers. Herald added subscribers: fedor.sergeev, jyknight. This patch (on top of https://reviews.llvm.org/D35755) provides the clang side necessary to enable the Solaris port of the sanitizers implemented by https://reviews.llvm.org/D40898, https

[PATCH] D40813: [clang-tidy] Adding Fuchsia checker for virtual inheritance

2017-12-06 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added inline comments. Comment at: test/clang-tidy/fuchsia-virtual-inheritance.cpp:30 +class D : public B, public C { + // CHECK-MESSAGES: [[@LINE-1]]:1: warning: virtual inheritance is disallowed [fuchsia-virtual-inheritance] + // CHECK-NEXT: class C : public B,

[PATCH] D40806: CodeGen: Fix invalid bitcasts for memcpy

2017-12-06 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm accepted this revision. arsenm added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D40806 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c

[PATCH] D40486: [clangd] Implemented logging using Context

2017-12-06 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. Mostly nits - throughout there's going to be lots of judgement calls about where to propagate context and where not. No need to get all those "exactly right", but trying to get a feel for what these answers are likely to be. Most of the interesting stuff is around the

[PATCH] D40897: [clangd] Introduce a "Symbol" class.

2017-12-06 Thread Marc-Andre Laperle via Phabricator via cfe-commits
malaperle added a comment. Hi! Have you looked into https://reviews.llvm.org/D40548 ? Maybe we need to coordinate the two a bit. Comment at: clangd/Symbol.h:37 +// The class presents a C++ symbol, e.g. class, function. +struct Symbol { + // The symbol identifier, using USR. -

[PATCH] D40901: Refactor lazy loading of template specializations. NFC

2017-12-06 Thread Vassil Vassilev via Phabricator via cfe-commits
v.g.vassilev updated this revision to Diff 125738. v.g.vassilev added a comment. Fix preexisting comment typo. https://reviews.llvm.org/D40901 Files: include/clang/AST/DeclTemplate.h lib/AST/DeclTemplate.cpp Index: lib/AST/DeclTemplate.cpp ==

[PATCH] D40895: Ignore pointers to incomplete types when diagnosing misaligned addresses

2017-12-06 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM, thank you! https://reviews.llvm.org/D40895 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-

[PATCH] D39812: [Driver, CodeGen] pass through and apply -fassociative-math

2017-12-06 Thread Sanjay Patel via Phabricator via cfe-commits
spatel added a comment. Ping * 3. https://reviews.llvm.org/D39812 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r319904 - Stringizing raw string literals containing newline

2017-12-06 Thread Taewook Oh via cfe-commits
Author: twoh Date: Wed Dec 6 09:00:53 2017 New Revision: 319904 URL: http://llvm.org/viewvc/llvm-project?rev=319904&view=rev Log: Stringizing raw string literals containing newline Summary: This patch implements 4.3 of http://open-std.org/jtc1/sc22/wg21/docs/papers/2014/n4220.pdf. If a raw stri

[PATCH] D39279: Stringizing raw string literals containing newline

2017-12-06 Thread Taewook Oh via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC319904: Stringizing raw string literals containing newline (authored by twoh). Changed prior to commit: https://reviews.llvm.org/D39279?vs=125564&id=125745#toc Repository: rC Clang https://reviews.l

[PATCH] D40909: [clang-format] Reorganize raw string delimiters

2017-12-06 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir created this revision. Herald added subscribers: cfe-commits, klimek. Repository: rC Clang https://reviews.llvm.org/D40909 Files: include/clang/Format/Format.h lib/Format/ContinuationIndenter.cpp lib/Format/ContinuationIndenter.h lib/Format/Format.cpp unittests/Format/Format

r319908 - [CUDA] Added overloads for '[unsigned] long' variants of shfl builtins.

2017-12-06 Thread Artem Belevich via cfe-commits
Author: tra Date: Wed Dec 6 09:40:35 2017 New Revision: 319908 URL: http://llvm.org/viewvc/llvm-project?rev=319908&view=rev Log: [CUDA] Added overloads for '[unsigned] long' variants of shfl builtins. Differential Revision: https://reviews.llvm.org/D40871 Modified: cfe/trunk/lib/Headers/__c

[PATCH] D40871: [CUDA] Added overloads for '[unsigned] long' variants of shfl builtins.

2017-12-06 Thread Artem Belevich via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC319908: [CUDA] Added overloads for '[unsigned] long' variants of shfl builtins. (authored by tra). Changed prior to commit: https://reviews.llvm.org/D40871?vs=125648&id=125756#toc Repository: rC Clan

[PATCH] D40871: [CUDA] Added overloads for '[unsigned] long' variants of shfl builtins.

2017-12-06 Thread Artem Belevich via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL319908: [CUDA] Added overloads for '[unsigned] long' variants of shfl builtins. (authored by tra). Changed prior to commit: https://reviews.llvm.org/D40871?vs=125648&id=125755#toc Repository: rL LLVM

r319909 - [NVPTX, CUDA] Added llvm.nvvm.fns intrinsic and matching __nvvm_fns builtin in clang.

2017-12-06 Thread Artem Belevich via cfe-commits
Author: tra Date: Wed Dec 6 09:50:05 2017 New Revision: 319909 URL: http://llvm.org/viewvc/llvm-project?rev=319909&view=rev Log: [NVPTX,CUDA] Added llvm.nvvm.fns intrinsic and matching __nvvm_fns builtin in clang. Differential Revision: https://reviews.llvm.org/D40872 Modified: cfe/trunk/

[PATCH] D40872: [NVPTX, CUDA] Added llvm.nvvm.fns intrinsic and matching __nvvm_fns builtin in clang.

2017-12-06 Thread Artem Belevich via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL319909: [NVPTX,CUDA] Added llvm.nvvm.fns intrinsic and matching __nvvm_fns builtin in… (authored by tra). Changed prior to commit: https://reviews.llvm.org/D40872?vs=125649&id=125757#toc Repository:

[PATCH] D40567: Always show template parameters in IR type names

2017-12-06 Thread Serge Pavlov via Phabricator via cfe-commits
sepavloff updated this revision to Diff 125761. sepavloff added a comment. Updated patch - old type names are generated always if --ir-type-names is not specified, - added new value of --ir-type-names, none, to suppress type names, - value of --ir-type-names is stored in module properties. Repo

Re: [PATCH] D40838: [OpenCL] Fix layering violation by getOpenCLTypeAddrSpace

2017-12-06 Thread David Blaikie via cfe-commits
Thanks! On Wed, Dec 6, 2017 at 2:12 AM Sven van Haastregt via Phabricator < revi...@reviews.llvm.org> wrote: > This revision was automatically updated to reflect the committed changes. > Closed by commit rC319883: [OpenCL] Fix layering violation by > getOpenCLTypeAddrSpace (authored by svenvh). >

Re: r319875 - Fix a bunch of wrong "tautological unsigned enum compare" diagnostics in C++.

2017-12-06 Thread Hans Wennborg via cfe-commits
This made Clang start warning about unsigned vs enum compares on Windows. $ echo 'enum E { foo }; bool f(unsigned a, E b) { return a == b; }' | bin/clang -Wsign-compare -c -x c++ - -target i686-pc-win32 :1:52: warning: comparison of integers of different signs: 'unsigned int' and 'E' [-Wsign-compa

LLVM buildmaster will be updated and restarted tonight

2017-12-06 Thread Galina Kistanova via cfe-commits
Hello everyone, LLVM buildmaster will be updated and restarted after 7 PM Pacific time. Thanks Galina ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D40911: [OpenMP] Fix PR35542: Correct adjusting of private reduction variable

2017-12-06 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld created this revision. Herald added a subscriber: cfe-commits. The adjustment is calculated with CreatePtrDiff() which returns the difference in (base) elements. This is passed to CreateGEP() so make sure that the GEP base has the correct pointer type: It needs to be a pointer to the base

[PATCH] D40911: [OpenMP] Fix PR35542: Correct adjusting of private reduction variable

2017-12-06 Thread Alexey Bataev via Phabricator via cfe-commits
ABataev added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:1110 +PrivateAddr.getPointer(), +SharedAddresses[N].first.getPointer()->getType()); +llvm::Value *Ptr = CGF.Builder.CreateGEP(PrivatePointer, Adjustment); Bette

[PATCH] D40897: [clangd] Introduce a "Symbol" class.

2017-12-06 Thread Haojian Wu via Phabricator via cfe-commits
hokein added a comment. In https://reviews.llvm.org/D40897#946708, @malaperle wrote: > Hi! Have you looked into https://reviews.llvm.org/D40548 ? Maybe we need to > coordinate the two a bit. Hi Marc! Thanks for the input! Yeah, Eric and I are working closely on a prototype of global code comp

[PATCH] D40911: [OpenMP] Fix PR35542: Correct adjusting of private reduction variable

2017-12-06 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:1110 +PrivateAddr.getPointer(), +SharedAddresses[N].first.getPointer()->getType()); +llvm::Value *Ptr = CGF.Builder.CreateGEP(PrivatePointer, Adjustment); ABat

[PATCH] D40911: [OpenMP] Fix PR35542: Correct adjusting of private reduction variable

2017-12-06 Thread Alexey Bataev via Phabricator via cfe-commits
ABataev added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:1110 +PrivateAddr.getPointer(), +SharedAddresses[N].first.getPointer()->getType()); +llvm::Value *Ptr = CGF.Builder.CreateGEP(PrivatePointer, Adjustment); Hahnf

[PATCH] D40911: [OpenMP] Fix PR35542: Correct adjusting of private reduction variable

2017-12-06 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld updated this revision to Diff 125768. Hahnfeld marked 3 inline comments as done. Hahnfeld added a comment. Get type from `Address`. https://reviews.llvm.org/D40911 Files: lib/CodeGen/CGOpenMPRuntime.cpp test/OpenMP/for_reduction_codegen.cpp test/OpenMP/for_reduction_codegen_UDR.c

[PATCH] D40911: [OpenMP] Fix PR35542: Correct adjusting of private reduction variable

2017-12-06 Thread Alexey Bataev via Phabricator via cfe-commits
ABataev accepted this revision. ABataev added a comment. This revision is now accepted and ready to land. LG https://reviews.llvm.org/D40911 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c

r319931 - Fix PR35542: Correct adjusting of private reduction variable

2017-12-06 Thread Jonas Hahnfeld via cfe-commits
Author: hahnfeld Date: Wed Dec 6 11:15:28 2017 New Revision: 319931 URL: http://llvm.org/viewvc/llvm-project?rev=319931&view=rev Log: Fix PR35542: Correct adjusting of private reduction variable The adjustment is calculated with CreatePtrDiff() which returns the difference in (base) elements. Th

[PATCH] D40911: [OpenMP] Fix PR35542: Correct adjusting of private reduction variable

2017-12-06 Thread Jonas Hahnfeld via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL319931: Fix PR35542: Correct adjusting of private reduction variable (authored by Hahnfeld). Changed prior to commit: https://reviews.llvm.org/D40911?vs=125768&id=125770#toc Repository: rL LLVM http

[PATCH] D40911: [OpenMP] Fix PR35542: Correct adjusting of private reduction variable

2017-12-06 Thread Jonas Hahnfeld via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC319931: Fix PR35542: Correct adjusting of private reduction variable (authored by Hahnfeld). Repository: rC Clang https://reviews.llvm.org/D40911 Files: lib/CodeGen/CGOpenMPRuntime.cpp test/OpenMP

r319942 - Delete special-case "out-of-range" handling for bools, and just use the normal

2017-12-06 Thread Richard Smith via cfe-commits
Author: rsmith Date: Wed Dec 6 11:23:19 2017 New Revision: 319942 URL: http://llvm.org/viewvc/llvm-project?rev=319942&view=rev Log: Delete special-case "out-of-range" handling for bools, and just use the normal codepath plus the new "minimum / maximum value of type" diagnostic to get the same eff

[PATCH] D39284: Allow conditions to be decomposed with structured bindings

2017-12-06 Thread Zhihao Yuan via Phabricator via cfe-commits
lichray added a comment. Can someone commit this please? One more patch then I'll go get a commit bit. Repository: rC Clang https://reviews.llvm.org/D39284 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mai

[PATCH] D40864: [Darwin] Add a new -mstack-probe option and enable by default

2017-12-06 Thread Akira Hatanaka via Phabricator via cfe-commits
ahatanak added inline comments. Comment at: lib/CodeGen/BackendUtil.cpp:442 Options.DebuggerTuning = CodeGenOpts.getDebuggerTuning(); + Options.EnableStackProbe = CodeGenOpts.StackProbe; aemerson wrote: > ahatanak wrote: > > Is there a reason you can't use

[PATCH] D40562: [Sema] Ignore decls in namespaces when global decls are not wanted.

2017-12-06 Thread Eric Liu via Phabricator via cfe-commits
ioeric updated this revision to Diff 125777. ioeric added a comment. - Add a new code-completion option IncludeNamespaceLevelDecls. For now, I only restrict this option work for qualified id completion to reduce the impact. Repository: rC Clang https://reviews.llvm.org/D40562 Files: inclu

[PATCH] D40562: [Sema] Ignore decls in namespaces when global decls are not wanted.

2017-12-06 Thread Eric Liu via Phabricator via cfe-commits
ioeric added a comment. In https://reviews.llvm.org/D40562#942521, @arphaman wrote: > In https://reviews.llvm.org/D40562#941753, @ilya-biryukov wrote: > > > In https://reviews.llvm.org/D40562#941570, @arphaman wrote: > > > > > I'm not actually 100% sure, but I would imagine that this one of the

[clang-tools-extra] r319948 - [CMake] Use PRIVATE in target_link_libraries for fuzzers.

2017-12-06 Thread Matt Morehouse via cfe-commits
Author: morehouse Date: Wed Dec 6 11:52:40 2017 New Revision: 319948 URL: http://llvm.org/viewvc/llvm-project?rev=319948&view=rev Log: [CMake] Use PRIVATE in target_link_libraries for fuzzers. Several fuzzers were missed by r319840. Modified: clang-tools-extra/trunk/clangd/fuzzer/CMakeLists

r319948 - [CMake] Use PRIVATE in target_link_libraries for fuzzers.

2017-12-06 Thread Matt Morehouse via cfe-commits
Author: morehouse Date: Wed Dec 6 11:52:40 2017 New Revision: 319948 URL: http://llvm.org/viewvc/llvm-project?rev=319948&view=rev Log: [CMake] Use PRIVATE in target_link_libraries for fuzzers. Several fuzzers were missed by r319840. Modified: cfe/trunk/tools/clang-format/fuzzer/CMakeLists.t

[PATCH] D40567: Always show template parameters in IR type names

2017-12-06 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith added a comment. In https://reviews.llvm.org/D40567#943747, @sepavloff wrote: > Although code generation (in LTO compilation) might be unaffected by this > distortions, other applications of IR linking suffer from it. It does not > allow to implement some checks, validation techniques an

[PATCH] D40819: Implement Attribute Target MultiVersioning (Improved edition!)

2017-12-06 Thread Erich Keane via Phabricator via cfe-commits
erichkeane marked 14 inline comments as done. erichkeane added a comment. Incoming patch! Comment at: include/clang/Basic/Attr.td:1809 bool DuplicateArchitecture = false; + bool operator ==(const ParsedTargetAttr &Other) { +return DuplicateArchitecture == Ot

r319950 - [clang] Use PRIVATE in target_link_libraries

2017-12-06 Thread Shoaib Meenai via cfe-commits
Author: smeenai Date: Wed Dec 6 12:05:42 2017 New Revision: 319950 URL: http://llvm.org/viewvc/llvm-project?rev=319950&view=rev Log: [clang] Use PRIVATE in target_link_libraries I'd missed this one in r319840 because I hadn't been configuring with an order file before. Modified: cfe/trunk/t

[PATCH] D40819: Implement Attribute Target MultiVersioning (Improved edition!)

2017-12-06 Thread Erich Keane via Phabricator via cfe-commits
erichkeane updated this revision to Diff 125779. erichkeane marked 3 inline comments as done. erichkeane added a comment. Fix all rsmith's and craig's fixes, AFAIK. https://reviews.llvm.org/D40819 Files: include/clang/AST/Decl.h include/clang/Basic/Attr.td include/clang/Basic/DiagnosticSe

Re: r319875 - Fix a bunch of wrong "tautological unsigned enum compare" diagnostics in C++.

2017-12-06 Thread Galina Kistanova via cfe-commits
Hello Richard, This commit broke the tests on the builder: http://lab.llvm.org:8011/builders/llvm-clang-x86_64-expensive-checks-win/builds/6598 . . . Failing Tests (1): Clang :: SemaCXX/warn-enum-compare.cpp Please have a look? Thanks Galina On Tue, Dec 5, 2017 at 7:00 PM, Richard Smith

[PATCH] D40548: [clangd] Prototyping index support and naive index-based global code completion. WIP

2017-12-06 Thread Eric Liu via Phabricator via cfe-commits
ioeric added a comment. Hi Marc, the patch is not ready for review yet. I am still cleaning up the prototype and will let you know when it's ready for review. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D40548 ___ cfe-commits mai

[libcxx] r319959 - [libc++] Create install-stripped targets

2017-12-06 Thread Shoaib Meenai via cfe-commits
Author: smeenai Date: Wed Dec 6 13:03:42 2017 New Revision: 319959 URL: http://llvm.org/viewvc/llvm-project?rev=319959&view=rev Log: [libc++] Create install-stripped targets LLVM is gaining install-*-stripped targets to perform stripped installs, and in order for this to be useful for install-di

[PATCH] D40680: [libc++] Create install-stripped targets

2017-12-06 Thread Shoaib Meenai via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL319959: [libc++] Create install-stripped targets (authored by smeenai). Repository: rL LLVM https://reviews.llvm.org/D40680 Files: libcxx/trunk/include/CMakeLists.txt libcxx/trunk/lib/CMakeLists.t

[PATCH] D38639: [clangd] #include statements support for Open definition

2017-12-06 Thread William Enright via Phabricator via cfe-commits
Nebiroth updated this revision to Diff 125813. Nebiroth added a comment. Herald added a subscriber: klimek. Using PPCallbacks interface to find non-preamble includes Created inner class to store vectors in to find locations Refactored methods to remove some unnecessary parameters Refactored Unit t

[PATCH] D40398: Remove ValueDependent assertions on ICE checks.

2017-12-06 Thread Matt Davis via Phabricator via cfe-commits
mattd added a comment. ping https://reviews.llvm.org/D40398 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D38639: [clangd] #include statements support for Open definition

2017-12-06 Thread William Enright via Phabricator via cfe-commits
Nebiroth updated this revision to Diff 125814. Nebiroth added a comment. Fixed re-added include Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D38639 Files: clangd/ClangdServer.cpp clangd/ClangdUnit.cpp clangd/ClangdUnit.h clangd/Protocol.h unittests/clangd/ClangdTests

[PATCH] D39375: [clang] Add PPCallbacks list to preprocessor when building a preacompiled preamble.

2017-12-06 Thread William Enright via Phabricator via cfe-commits
Nebiroth updated this revision to Diff 125815. Nebiroth added a comment. Reverted formatting changes to ASTUnit Repository: rC Clang https://reviews.llvm.org/D39375 Files: include/clang/Frontend/PrecompiledPreamble.h lib/Frontend/ASTUnit.cpp lib/Frontend/PrecompiledPreamble.cpp Index

[PATCH] D40637: [CMake] Support runtimes and monorepo layouts when looking for libc++

2017-12-06 Thread Chris Bieneman via Phabricator via cfe-commits
beanz accepted this revision. beanz added a comment. This revision is now accepted and ready to land. LGTM Repository: rCRT Compiler Runtime https://reviews.llvm.org/D40637 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm

[PATCH] D40815: [libcxxabi] Use the correct variable name for target triple in lit

2017-12-06 Thread Chris Bieneman via Phabricator via cfe-commits
beanz accepted this revision. beanz added a comment. This revision is now accepted and ready to land. LGTM. Repository: rCXXA libc++abi https://reviews.llvm.org/D40815 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/

[PATCH] D40814: [libcxx] Use the correct variable name for target triple in lit

2017-12-06 Thread Chris Bieneman via Phabricator via cfe-commits
beanz accepted this revision. beanz added a comment. This revision is now accepted and ready to land. LGTM. Repository: rCXX libc++ https://reviews.llvm.org/D40814 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-

[PATCH] D40818: [libcxxabi] Pass LIBCXXABI_SYSROOT and LIBCXXABI_GCC_TOOLCHAIN to lit

2017-12-06 Thread Chris Bieneman via Phabricator via cfe-commits
beanz accepted this revision. beanz added a comment. This revision is now accepted and ready to land. LGTM Repository: rCXXA libc++abi https://reviews.llvm.org/D40818 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/c

[PATCH] D38110: [libunwind][MIPS]: Add support for unwinding in O32 and N64 processes.

2017-12-06 Thread Saleem Abdulrasool via Phabricator via cfe-commits
compnerd accepted this revision. compnerd added a comment. LGTM if @sdardis is good with it https://reviews.llvm.org/D38110 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r319983 - [clang] Add PRIVATE to target_link_libraries

2017-12-06 Thread Shoaib Meenai via cfe-commits
Author: smeenai Date: Wed Dec 6 15:02:00 2017 New Revision: 319983 URL: http://llvm.org/viewvc/llvm-project?rev=319983&view=rev Log: [clang] Add PRIVATE to target_link_libraries Another follow-up to r319840. I'd done a test configure with LLVM_BUILD_STATIC, so I'm not sure why this didn't show u

r319986 - [Lex] Fix some Clang-tidy modernize and Include What You Use warnings; other minor fixes (NFC).

2017-12-06 Thread Eugene Zelenko via cfe-commits
Author: eugenezelenko Date: Wed Dec 6 15:18:41 2017 New Revision: 319986 URL: http://llvm.org/viewvc/llvm-project?rev=319986&view=rev Log: [Lex] Fix some Clang-tidy modernize and Include What You Use warnings; other minor fixes (NFC). Modified: cfe/trunk/include/clang/Lex/HeaderSearch.h

[PATCH] D40925: Add option -fkeep-static-consts

2017-12-06 Thread Elizabeth Andrews via Phabricator via cfe-commits
eandrews created this revision. Currently clang does not emit unused static constants declared globally. Local static constants are emitted by default. -fkeep-static-consts can be used to emit static constants declared globally even if they are not used. This could be useful for producing ident

[PATCH] D40929: Unblock Swift Calling Convention Mangling on Windows

2017-12-06 Thread Thomas Roughton via Phabricator via cfe-commits
troughton created this revision. troughton added a project: clang. Herald added a subscriber: cfe-commits. Following discussion on the [[ https://lists.swift.org/pipermail/swift-dev/Week-of-Mon-20171204/006123.html | swift-dev mailing list] and on a pull request for swift-clang

[PATCH] D40380: Remove old concepts parsing code

2017-12-06 Thread Hubert Tong via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC319992: Remove old concepts parsing code (authored by hubert.reinterpretcast). Changed prior to commit: https://reviews.llvm.org/D40380?vs=124032&id=125844#toc Repository: rC Clang https://reviews.l

r319992 - Remove old concepts parsing code

2017-12-06 Thread Hubert Tong via cfe-commits
Author: hubert.reinterpretcast Date: Wed Dec 6 16:34:20 2017 New Revision: 319992 URL: http://llvm.org/viewvc/llvm-project?rev=319992&view=rev Log: Remove old concepts parsing code Summary: This is so we can implement concepts per P0734R0. Relevant failing test cases are disabled. Reviewers: hu

[PATCH] D40929: Unblock Swift Calling Convention Mangling on Windows

2017-12-06 Thread Shoaib Meenai via Phabricator via cfe-commits
smeenai added a comment. Patch is missing context. Comment at: lib/AST/MicrosoftMangle.cpp:2133 + llvm::errs() << "Unsupported CC for mangling: " << CC << ".\n"; case CC_Win64: case CC_X86_64SysV: You still need the default label, right? Reposi

[PATCH] D38110: [libunwind][MIPS]: Add support for unwinding in O32 and N64 processes.

2017-12-06 Thread Simon Dardis via Phabricator via cfe-commits
sdardis accepted this revision. sdardis added a comment. This revision is now accepted and ready to land. LGTM. https://reviews.llvm.org/D38110 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cf

[libcxx] r319994 - [libcxx] [test] Strip trailing whitespace. NFC.

2017-12-06 Thread Stephan T. Lavavej via cfe-commits
Author: stl_msft Date: Wed Dec 6 16:50:23 2017 New Revision: 319994 URL: http://llvm.org/viewvc/llvm-project?rev=319994&view=rev Log: [libcxx] [test] Strip trailing whitespace. NFC. Modified: libcxx/trunk/test/std/experimental/filesystem/class.path/path.member/path.decompose/path.decompose.

[PATCH] D40929: Unblock Swift Calling Convention Mangling on Windows

2017-12-06 Thread Thomas Roughton via Phabricator via cfe-commits
troughton updated this revision to Diff 125847. troughton added a comment. Add back a mistakenly removed default case. Repository: rC Clang https://reviews.llvm.org/D40929 Files: lib/AST/MicrosoftMangle.cpp Index: lib/AST/MicrosoftMangle.cpp ==

[PATCH] D40560: [analyzer] WIP: Get construction into `operator new` running in simple cases.

2017-12-06 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ updated this revision to Diff 125850. NoQ added a comment. Replaced the live expression hack with a slightly better approach. It doesn't update the live variables analysis to take `CFGNewAllocator` into account, but at least tests now pass. In order to keep the return value produced by the

  1   2   >