[PATCH] D46836: Fix some rtti-options tests

2018-05-15 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC332384: Fixed some rtti-options tests. (authored by ssrivastava, committed by ). Changed prior to commit: https://reviews.llvm.org/D46836?vs=146632&id=146893#toc Repository: rC Clang https://reviews

[PATCH] D41968: [libunwind][MIPS] Support MIPS floating-point registers for hard-float ABIs.

2018-05-15 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL332414: [libunwind][MIPS] Support MIPS floating-point registers for hard-float ABIs. (authored by jhb, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.

[PATCH] D46902: [analyzer] Make plist-html multi-file.

2018-05-15 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL332417: [analyzer] Make plist-html diagnostic consumer produce multi-file reports. (authored by dergachev, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://revie

[PATCH] D46001: [CodeComplete] Expose helpers to get RawComment of completion result.

2018-05-16 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC332457: [CodeComplete] Expose helpers to get RawComment of completion result. (authored by ibiryukov, committed by ). Changed prior to commit: https://reviews.llvm.org/D46001?vs=146812&id=147061#toc Re

[PATCH] D46000: [AST] Added a helper to extract a user-friendly text of a comment.

2018-05-16 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL332458: [AST] Added a helper to extract a user-friendly text of a comment. (authored by ibiryukov, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/

[PATCH] D46002: [clangd] Parse all comments in Sema and completion.

2018-05-16 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rCTE332460: [clangd] Parse all comments in Sema and completion. (authored by ibiryukov, committed by ). Changed prior to commit: https://reviews.llvm.org/D46002?vs=146831&id=147064#toc Repository: rCTE

[PATCH] D45999: [clangd] Retrieve minimally formatted comment text in completion.

2018-05-16 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rCTE332459: [clangd] Retrieve minimally formatted comment text in completion. (authored by ibiryukov, committed by ). Changed prior to commit: https://reviews.llvm.org/D45999?vs=147060&id=147063#toc Repo

[PATCH] D44934: [analyzer] Improve the modeling of `memset()`.

2018-05-16 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL332463: [analyzer] Improve the modeling of memset(). (authored by henrywong, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D44934?vs=146816&i

[PATCH] D46936: [Timers] TimerGroup::printJSONValues(): print mem timer with .mem suffix

2018-05-16 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL332503: [Timers] TimerGroup::printJSONValues(): print mem timer with .mem suffix (authored by lebedevri, committed by ). Changed prior to commit: https://reviews.llvm.org/D46936?vs=147052&id=147137#toc

[PATCH] D46937: [Timers] TimerGroup::printJSONValue(): print doubles with no precision loss

2018-05-16 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL332504: [Timers] TimerGroup::printJSONValue(): print doubles with no precision loss (authored by lebedevri, committed by ). Changed prior to commit: https://reviews.llvm.org/D46937?vs=147053&id=147138#t

[PATCH] D46938: [Timers] TimerGroup: make printJSONValues() method public

2018-05-16 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL332505: [Timers] TimerGroup: make printJSONValues() method public (authored by lebedevri, committed by ). Changed prior to commit: https://reviews.llvm.org/D46938?vs=147054&id=147139#toc Repository:

[PATCH] D46939: [Timers] TimerGroup: add constructor from StringMap

2018-05-16 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL332506: [Timers] TimerGroup: add constructor from StringMap (authored by lebedevri, committed by ). Changed prior to commit: https://reviews.llvm.org/D46939?vs=147055&id=147140#toc Reposito

[PATCH] D46971: [DWARF] Get RA from RA register even if it appears unused

2018-05-16 Thread whitequark via Phabricator via cfe-commits
whitequark created this revision. whitequark added a reviewer: compnerd. Herald added a subscriber: JDevlieghere. If prolog info lists the RA register as unused, it means that the current stack frame corresponds to a leaf function which never needs to save the RA register. The RA register is of co

[PATCH] D46971: [DWARF] Get RA from RA register even if it appears unused

2018-05-16 Thread whitequark via Phabricator via cfe-commits
whitequark updated this revision to Diff 147153. whitequark added a comment. Reuploaded diff with context. Repository: rUNW libunwind https://reviews.llvm.org/D46971 Files: src/DwarfInstructions.hpp Index: src/DwarfInstructions.hpp

[PATCH] D43424: [clang-doc] Implement a (simple) Markdown generator

2018-05-17 Thread Joe via Phabricator via cfe-commits
rja added inline comments. Comment at: clang-doc/generators/MDGenerator.cpp:57 + sys::path::native(NamespacesPath, Path); + // for (const auto &Namespace : I->Namespace) + // sys::path::append(Path, IS->find(Namespace)->Name); remove commented code? Reposi

[PATCH] D46982: Do not enable RTTI with -fexceptions, for PS4

2018-05-18 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL332784: Do not enable RTTI with -fexceptions, for PS4 (authored by ssrivastava, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D46982?vs=14719

[PATCH] D47097: [WIP][DebugInfo] Preserve scope in auto generated StoreInst

2018-05-18 Thread Anastasis via Phabricator via cfe-commits
gramanas created this revision. gramanas added a reviewer: vsk. Herald added subscribers: JDevlieghere, aprantl. In this test there is a store instruction generated by clang for the function argument `int b` where the debug info is missing. The goal of this patch is to instruct clang to add an ar

[PATCH] D41785: Print the qualified name when dumping deserialized decls.

2018-05-20 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL332817: Print the qualified name when dumping deserialized decls. (authored by vvassilev, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D4178

[PATCH] D47029: [X86] Remove some preprocessor feature checks from intrinsic headers

2018-05-20 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC332830: [X86] Remove some preprocessor feature checks from intrinsic headers (authored by ctopper, committed by ). Repository: rC Clang https://reviews.llvm.org/D47029 Files: lib/Headers/prfchwintri

[PATCH] D47097: [WIP][DebugInfo] Preserve scope in auto generated StoreInst

2018-05-21 Thread Anastasis via Phabricator via cfe-commits
gramanas updated this revision to Diff 147838. gramanas added a comment. - Apply debug location Repository: rC Clang https://reviews.llvm.org/D47097 Files: lib/CodeGen/CGDecl.cpp test/CodeGen/debug-info-preserve-scope.c Index: test/CodeGen/debug-info-preserve-scope.c ==

[PATCH] D47097: [WIP][DebugInfo] Preserve scope in auto generated StoreInst

2018-05-21 Thread Anastasis via Phabricator via cfe-commits
gramanas updated this revision to Diff 147840. gramanas marked an inline comment as done. gramanas added a comment. Update according to the comments Repository: rC Clang https://reviews.llvm.org/D47097 Files: lib/CodeGen/CGDecl.cpp test/CodeGen/debug-info-preserve-scope.c Index: test/C

[PATCH] D46863: [X86] Use __builtin_convertvector to implement some of the packed integer to packed float conversion intrinsics.

2018-05-21 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC332882: [X86] Use __builtin_convertvector to implement some of the packed integer to… (authored by ctopper, committed by ). Changed prior to commit: https://reviews.llvm.org/D46863?vs=146750&id=147853#t

[PATCH] D47125: [X86] Remove masking from pternlog llvm intrinsics and use a select instruction instead.

2018-05-21 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL332891: [X86] Remove masking from pternlog llvm intrinsics and use a select instruction… (authored by ctopper, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://revie

[PATCH] D47161: update

2018-05-21 Thread Anastasis via Phabricator via cfe-commits
gramanas created this revision. Herald added a subscriber: cfe-commits. Repository: rC Clang https://reviews.llvm.org/D47161 Files: lib/CodeGen/CGDecl.cpp test/CodeGen/debug-info-preserve-scope.c Index: test/CodeGen/debug-info-preserve-scope.c

[PATCH] D47097: [DebugInfo] Preserve scope in auto generated StoreInst

2018-05-21 Thread Anastasis via Phabricator via cfe-commits
gramanas updated this revision to Diff 147876. gramanas added a comment. Update diff Repository: rC Clang https://reviews.llvm.org/D47097 Files: lib/CodeGen/CGDecl.cpp test/CodeGen/debug-info-preserve-scope.c Index: test/CodeGen/debug-info-preserve-scope.c =

[PATCH] D47066: [clangd] Remove ignored Preamble::CanReuse call from completion

2018-05-22 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL332976: [clangd] Remove ignored Preamble::CanReuse call from completion (authored by ibiryukov, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/D47

[PATCH] D47161: update

2018-05-22 Thread Anastasis via Phabricator via cfe-commits
gramanas abandoned this revision. gramanas added a comment. Duplicate of https://reviews.llvm.org/D47097 Repository: rC Clang https://reviews.llvm.org/D47161 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/ma

[PATCH] D47097: [DebugInfo] Preserve scope in auto generated StoreInst

2018-05-22 Thread Anastasis via Phabricator via cfe-commits
gramanas updated this revision to Diff 148034. gramanas added a comment. Move ApplyDebugLocation before CreateMemTemp Repository: rC Clang https://reviews.llvm.org/D47097 Files: lib/CodeGen/CGDecl.cpp test/CodeGen/debug-info-preserve-scope.c Index: test/CodeGen/debug-info-preserve-scop

[PATCH] D47174: [X86] Move 128-bit f16c intrinsics to __emmintrin_f16c.h include from emmintrin.h. Move 256-bit f16c intrinsics back to f16cintrin.h

2018-05-22 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL333014: [X86] Move 128-bit f16c intrinsics to __emmintrin_f16c.h include from emmintrin. (authored by ctopper, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://r

[PATCH] D47161: update

2018-05-23 Thread Anastasis via Phabricator via cfe-commits
gramanas added a comment. It was a mistake on my part with the arcanist tool. The documentation is fine, the only problems I encountered were due to my ignorance of how these systems work. Repository: rC Clang https://reviews.llvm.org/D47161 __

[PATCH] D47097: [DebugInfo] Preserve scope in auto generated StoreInst

2018-05-23 Thread Anastasis via Phabricator via cfe-commits
gramanas updated this revision to Diff 148198. gramanas added a comment. Set debug location to the entry block alloca Repository: rC Clang https://reviews.llvm.org/D47097 Files: lib/CodeGen/CGDecl.cpp lib/CodeGen/CGExpr.cpp test/CodeGen/debug-info-preserve-scope.c Index: test/CodeGen

[PATCH] D47097: [DebugInfo] Preserve scope in auto generated StoreInst

2018-05-23 Thread Anastasis via Phabricator via cfe-commits
gramanas updated this revision to Diff 148200. gramanas marked an inline comment as done. gramanas added a comment. Remove redundant `this` Repository: rC Clang https://reviews.llvm.org/D47097 Files: lib/CodeGen/CGDecl.cpp lib/CodeGen/CGExpr.cpp test/CodeGen/debug-info-preserve-scope.c

[PATCH] D47182: [X86] Move all Intel defined intrinsic includes into immintrin.h

2018-05-23 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL333110: [X86] Move all Intel defined intrinsic includes into immintrin.h (authored by ctopper, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/

[PATCH] D46971: [DWARF] Get RA from RA register even if it appears unused

2018-05-23 Thread whitequark via Phabricator via cfe-commits
whitequark added a comment. @compnerd ping Repository: rUNW libunwind https://reviews.llvm.org/D46971 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D45920: [analyzer] Move RangeSet related declarations into the RangedConstraintManager header.

2018-05-24 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC333179: [analyzer] Move RangeSet related declarations into the RangedConstraintManager… (authored by mramalho, committed by ). Repository: rC Clang https://reviews.llvm.org/D45920 Files: lib/StaticA

[PATCH] D47097: [DebugInfo] Preserve scope in auto generated StoreInst

2018-05-24 Thread Anastasis via Phabricator via cfe-commits
gramanas updated this revision to Diff 148414. gramanas added a comment. Adress review comments. Limit changes to the storeInst Repository: rC Clang https://reviews.llvm.org/D47097 Files: lib/CodeGen/CGDecl.cpp test/CodeGen/debug-info-preserve-scope.c Index: test/CodeGen/debug-info-pre

[PATCH] D47274: [clangd] Serve comments for headers decls from dynamic index only

2018-05-24 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL333189: [clangd] Serve comments for headers decls from dynamic index only (authored by ibiryukov, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/D

[PATCH] D47272: [clangd] Build index on preamble changes instead of the AST changes

2018-05-24 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rCTE333196: [clangd] Build index on preamble changes instead of the AST changes (authored by ibiryukov, committed by ). Changed prior to commit: https://reviews.llvm.org/D47272?vs=148421&id=148424#toc Re

[PATCH] D47097: [DebugInfo] Preserve scope in auto generated StoreInst

2018-05-24 Thread Anastasis via Phabricator via cfe-commits
gramanas added a comment. In https://reviews.llvm.org/D47097#149, @probinson wrote: > Can we step back a second and better explain what the problem is? With > current Clang the debug info for this function looks okay to me. > The store that is "missing" a debug location is homing the formal

[PATCH] D47097: [DebugInfo] Preserve scope in auto generated StoreInst

2018-05-25 Thread Anastasis via Phabricator via cfe-commits
gramanas updated this revision to Diff 148588. gramanas added a comment. I added a test that illustrates what @vsk is talking about. Sorry for not providing the relevant information but I thought this would be a simple one liner like https://reviews.llvm.org/rL327800. I will update the revision'

[PATCH] D41881: [analyzer] Flag bcmp, bcopy and bzero as obsolete

2018-05-25 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC26: [analyzer] Add security checks for bcmp(), bcopy(), bzero(). (authored by dergachev, committed by ). Repository: rC Clang https://reviews.llvm.org/D41881 Files: include/clang/StaticAnalyzer/

[PATCH] D47331: [clangd] Remove accessors for top-level decls from preamble

2018-05-28 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rCTE71: [clangd] Remove accessors for top-level decls from preamble (authored by ibiryukov, committed by ). Changed prior to commit: https://reviews.llvm.org/D47331?vs=148605&id=148806#toc Repository

[PATCH] D44888: [RISCV] Add -mrelax/-mno-relax flags to enable/disable RISCV linker relaxation

2018-05-28 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL85: [RISCV] Add -mrelax/-mno-relax flags to enable/disable RISCV linker relaxation (authored by shiva, committed by ). Changed prior to commit: https://reviews.llvm.org/D44888?vs=148548&id=148840#to

[PATCH] D46971: [DWARF] Get RA from RA register even if it appears unused

2018-05-29 Thread whitequark via Phabricator via cfe-commits
whitequark added a comment. > It would be nice if we had a test case added for this This is inherently architecture and platform specific, and libunwind currently doesn't have any provisions for this... In principle, would a x86-only Linux-only test suffice? I'm not sure if I can spend time mak

[PATCH] D47515: [clang-format] Process line's children once in guessIsObjC

2018-05-30 Thread Attila via Phabricator via cfe-commits
Uran198 created this revision. Uran198 added reviewers: djasper, klimek. Repository: rC Clang https://reviews.llvm.org/D47515 Files: lib/Format/Format.cpp Index: lib/Format/Format.cpp === --- lib/Format/Format.cpp +++ lib/Form

[PATCH] D47476: Support __iso_volatile_load8 etc on aarch64-win32.

2018-05-30 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC333513: Support __iso_volatile_load8 etc on aarch64-win32. (authored by statham, committed by ). Changed prior to commit: https://reviews.llvm.org/D47476?vs=148900&id=149055#toc Repository: rC Clang

[PATCH] D47476: Support __iso_volatile_load8 etc on aarch64-win32.

2018-05-30 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL333513: Support __iso_volatile_load8 etc on aarch64-win32. (authored by statham, committed by ). Herald added a subscriber: llvm-commits. Repository: rC Clang https://reviews.llvm.org/D47476 Files:

[PATCH] D47519: [clang-format] Detect amp type as TT_PointerOrReference in function annotations

2018-05-30 Thread Attila via Phabricator via cfe-commits
Uran198 created this revision. Uran198 added reviewers: klimek, krasimir, djasper. Repository: rC Clang https://reviews.llvm.org/D47519 Files: lib/Format/TokenAnnotator.cpp unittests/Format/FormatTest.cpp Index: unittests/Format/FormatTest.cpp

[PATCH] D47520: [clang-format] Allow break between question and lambda

2018-05-30 Thread Attila via Phabricator via cfe-commits
Uran198 created this revision. Uran198 added reviewers: djasper, klimek. Before the change clang-format would break and make no reformatting. Repository: rC Clang https://reviews.llvm.org/D47520 Files: lib/Format/TokenAnnotator.cpp unittests/Format/FormatTest.cpp Index: unittests/Forma

[PATCH] D47521: [clang-format] Fix parsing lambdas with noexcept

2018-05-30 Thread Attila via Phabricator via cfe-commits
Uran198 created this revision. Uran198 added reviewers: klimek, djasper. Repository: rC Clang https://reviews.llvm.org/D47521 Files: lib/Format/UnwrappedLineParser.cpp unittests/Format/FormatTest.cpp Index: unittests/Format/FormatTest.cpp =

[PATCH] D38231: fixing a bug in mask[z]_set1

2017-09-25 Thread jina via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314102: fixing a bug in mask[z]_set1 intrinsic (authored by jina.nahias). Changed prior to commit: https://reviews.llvm.org/D38231?vs=116513&id=116545#toc Repository: rL LLVM https://reviews.llvm.or

[PATCH] D38225: [clangd] Fix missing "message" key when responding with unsupported method

2017-09-25 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314119: [clangd] Fix missing "message" key when responding with unsupported method (authored by d0k). Repository: rL LLVM https://reviews.llvm.org/D38225 Files: clang-tools-extra/trunk/clangd/JSONRP

[PATCH] D36737: [analyzer] Store design discussions in docs/analyzer for future use.

2017-09-26 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314218: [analyzer] Keep track of design discusions as part of analyzer documentation. (authored by dergachev). Changed prior to commit: https://reviews.llvm.org/D36737?vs=116520&id=116724#toc Repositor

[PATCH] D37023: [analyzer] Fix bugreporter::getDerefExpr() again.

2017-09-27 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314287: [analyzer] Fix and refactor bugreporter::getDerefExpr() API. (authored by dergachev). Changed prior to commit: https://reviews.llvm.org/D37023?vs=116673&id=116781#toc Repository: rL LLVM htt

[PATCH] D37025: [analyzer] Support more pointer arithmetic in bugreporter::getDerefExpr().

2017-09-27 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314290: [analyzer] Match more patterns in bugreporter::getDerefExpr() API. (authored by dergachev). Changed prior to commit: https://reviews.llvm.org/D37025?vs=112221&id=116783#toc Repository: rL LLV

[PATCH] D37804: [OpenCL] Handle address space conversion while setting type alignment

2017-09-27 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314304: [OpenCL] Handle address space conversion while setting type alignment. (authored by stulova). Changed prior to commit: https://reviews.llvm.org/D37804?vs=116548&id=116809#toc Repository: rL L

[PATCH] D38356: keeps include path order when emulate msvc compiler

2017-09-28 Thread comicfans44 via Phabricator via cfe-commits
comicfans44 created this revision. comicfans44 added a project: clang. clang will drop user supplied include path if it's already in system include paths to emulate gcc. if two files with same name both exist in user path and system path, and user set system path first in arguments, this leads d

[PATCH] D38083: [clangd] Skip informative qualifier chunks.

2017-09-28 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314445: [clangd] Skip informative qualifier chunks. (authored by ibiryukov). Repository: rL LLVM https://reviews.llvm.org/D38083 Files: clang-tools-extra/trunk/clangd/ClangdUnit.cpp clang-tools-ex

[PATCH] D37413: [X86][MS-InlineAsm] Extended support for variables / identifiers on memory / immediate expressions

2017-09-29 Thread coby via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314494: [X86][MS-InlineAsm] Extended support for variables / identifiers on memory /… (authored by coby). Changed prior to commit: https://reviews.llvm.org/D37413?vs=115564&id=117095#toc Repository:

[PATCH] D37903: Fix assume-filename handling in clang-format.el

2017-09-29 Thread Philipp via Phabricator via cfe-commits
phst added inline comments. Comment at: tools/clang-format/clang-format.el:122 ;;;###autoload -(defun clang-format-region (start end &optional style assume-file-name) - "Use clang-format to format the code between START and END according to STYLE -using ASSUME-FILE-NAME to loca

[PATCH] D37624: add support for -fno-instrument-functions and -finstrument-functions-exclude-{file, function}-list= to match gcc options.

2017-09-30 Thread kchoi via Phabricator via cfe-commits
choikwa updated this revision to Diff 117264. choikwa added a comment. Addressing Hal's feedback https://reviews.llvm.org/D37624 Files: docs/ClangCommandLineReference.rst include/clang/Driver/Options.td include/clang/Frontend/CodeGenOptions.h lib/CodeGen/CodeGenFunction.cpp lib/CodeGe

[PATCH] D37624: add support for -fno-instrument-functions and -finstrument-functions-exclude-{file, function}-list= to match gcc options.

2017-09-30 Thread kchoi via Phabricator via cfe-commits
choikwa updated this revision to Diff 117267. choikwa added a comment. - - Address formating feedback, remove redundant inline https://reviews.llvm.org/D37624 Files: docs/ClangCommandLineReference.rst include/clang/Driver/Options.td include/clang/Frontend/CodeGenOptions.h lib/CodeGen/Co

[PATCH] D37624: add support for -fno-instrument-functions and -finstrument-functions-exclude-{file, function}-list= to match gcc options.

2017-09-30 Thread kchoi via Phabricator via cfe-commits
choikwa updated this revision to Diff 117268. choikwa added a comment. - add comment to CPP test to explain usage https://reviews.llvm.org/D37624 Files: docs/ClangCommandLineReference.rst include/clang/Driver/Options.td include/clang/Frontend/CodeGenOptions.h lib/CodeGen/CodeGenFunction

[PATCH] D37624: add support for -fno-instrument-functions and -finstrument-functions-exclude-{file, function}-list= to match gcc options.

2017-09-30 Thread kchoi via Phabricator via cfe-commits
choikwa updated this revision to Diff 117269. choikwa added a comment. - add more CPP tests: func overload, template special https://reviews.llvm.org/D37624 Files: docs/ClangCommandLineReference.rst include/clang/Driver/Options.td include/clang/Frontend/CodeGenOptions.h lib/CodeGen/Code

[PATCH] D37624: add support for -fno-instrument-functions and -finstrument-functions-exclude-{file, function}-list= to match gcc options.

2017-09-30 Thread kchoi via Phabricator via cfe-commits
choikwa added a comment. In https://reviews.llvm.org/D37624#885290, @hfinkel wrote: > In https://reviews.llvm.org/D37624#885288, @choikwa wrote: > > > - add comment to CPP test to explain usage > > > Thanks. Please also add some tests showing matching overloaded functions, > functions with templ

[PATCH] D37624: add support for -fno-instrument-functions and -finstrument-functions-exclude-{file, function}-list= to match gcc options.

2017-10-01 Thread kchoi via Phabricator via cfe-commits
choikwa added a comment. In https://reviews.llvm.org/D37624#885308, @hfinkel wrote: > In https://reviews.llvm.org/D37624#885295, @choikwa wrote: > > > In https://reviews.llvm.org/D37624#885290, @hfinkel wrote: > > > > > In https://reviews.llvm.org/D37624#885288, @choikwa wrote: > > > > > > > - ad

[PATCH] D38445: [x86][inlin-asm] allow recognition of MPX regs inside ms inline-asm blob

2017-10-01 Thread coby via Phabricator via cfe-commits
coby created this revision. Herald added a subscriber: eraman. Repository: rL LLVM https://reviews.llvm.org/D38445 Files: lib/Basic/Targets/X86.cpp test/CodeGen/ms-inline-asm.c Index: lib/Basic/Targets/X86.cpp === --- lib/Ba

[PATCH] D37466: D37461: fixups for existing InlineAsm tests + adding new ones

2017-10-01 Thread coby via Phabricator via cfe-commits
coby added inline comments. Comment at: test/CodeGen/ms-inline-asm.cpp:37-38 - int lvar = 10; - __asm mov eax, offset Foo::ptr - __asm mov eax, offset Foo::Bar::ptr -// CHECK-LABEL: define void @_Z2t2v() rnk wrote: > These don't seem tested anywhere now I've t

[PATCH] D37624: add support for -fno-instrument-functions and -finstrument-functions-exclude-{file, function}-list= to match gcc options.

2017-10-01 Thread kchoi via Phabricator via cfe-commits
choikwa added a comment. > Can you get more information on what GCC actually implemented and why? It's > not clear to me that ignoring the namespaces is the most-useful way to do > this. I don't want to emulate GCC bugs, but maybe there's a good reason why > their implementation works this way.

[PATCH] D37150: [clangd] Command line arg to specify compile_commands.json path

2017-10-02 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314678: [clangd] Command line arg to specify compile_commands.json path (authored by ibiryukov). Changed prior to commit: https://reviews.llvm.org/D37150?vs=117340&id=117354#toc Repository: rL LLVM

[PATCH] D37544: [ubsan] Skip alignment checks which are folded away

2017-10-02 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314752: [ubsan] Skip alignment checks which are folded away (authored by vedantk). Changed prior to commit: https://reviews.llvm.org/D37544?vs=116374&id=117461#toc Repository: rL LLVM https://review

[PATCH] D37903: Fix assume-filename handling in clang-format.el

2017-10-03 Thread Philipp via Phabricator via cfe-commits
phst accepted this revision. phst added inline comments. Comment at: tools/clang-format/clang-format.el:126 +no active region. If no style is given uses `clang-format-style'. Use +ASSUME-FILE-NAME to locate a style config file, if no assume-file-name is given +uses the function

[PATCH] D38502: allow access decl without using keyword when emulate msvc

2017-10-03 Thread comicfans44 via Phabricator via cfe-commits
comicfans44 created this revision. comicfans44 added a project: clang. msvc allow such code construction in vs2010 / vs2017 , needs someone check this behavior in VS2012/VS2013/VS2015 Repository: rL LLVM https://reviews.llvm.org/D38502 Files: clang/lib/Sema/SemaDeclCXX.cpp Index: clang/

[PATCH] D38483: [ExprConstant] Allow constexpr ctor to modify non static data members in body

2017-10-03 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314865: [ExprConstant] Allow constexpr ctor to modify non static data members (authored by epilk). Changed prior to commit: https://reviews.llvm.org/D38483?vs=117529&id=117605#toc Repository: rL LLVM

[PATCH] D38358: [analyzer] Fix autodetection of getSVal()'s type argument.

2017-10-04 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314910: [analyzer] Fix autodetection of binding types. (authored by dergachev). Changed prior to commit: https://reviews.llvm.org/D38358?vs=117360&id=117681#toc Repository: rL LLVM https://reviews.l

[PATCH] D35216: [analyzer] Escape symbols when creating std::initializer_list.

2017-10-05 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314975: [analyzer] Fix leak false positives on stuff put in C++/ObjC initializer lists. (authored by dergachev). Changed prior to commit: https://reviews.llvm.org/D35216?vs=117699&id=117785#toc Reposit

[PATCH] D38583: [clangd] Added async API to run code completion.

2017-10-05 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314989: [clangd] Added async API to run code completion. (authored by ibiryukov). Repository: rL LLVM https://reviews.llvm.org/D38583 Files: clang-tools-extra/trunk/clangd/ClangdLSPServer.cpp clan

[PATCH] D38048: [clangd] Add textDocument/signatureHelp

2017-10-06 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315055: [clangd] Add textDocument/signatureHelp (authored by ibiryukov). Changed prior to commit: https://reviews.llvm.org/D38048?vs=117580&id=117983#toc Repository: rL LLVM https://reviews.llvm.org

[PATCH] D38672: lowering shuffle f/i intrinsic - clang part

2017-10-08 Thread jina via Phabricator via cfe-commits
jina.nahias created this revision. https://reviews.llvm.org/D38672 Files: lib/Headers/avx512fintrin.h lib/Headers/avx512vlintrin.h test/CodeGen/avx512f-builtins.c test/CodeGen/avx512vl-builtins.c Index: test/CodeGen/avx512vl-builtins.c

[PATCH] D38672: [X86][AVX512] lowering shuffle f/i intrinsic - clang part

2017-10-08 Thread jina via Phabricator via cfe-commits
jina.nahias updated this revision to Diff 118166. https://reviews.llvm.org/D38672 Files: lib/Headers/avx512fintrin.h lib/Headers/avx512vlintrin.h test/CodeGen/avx512f-builtins.c test/CodeGen/avx512vl-builtins.c Index: test/CodeGen/avx512vl-builtins.c =

[PATCH] D38683: [X86][AVX512] lowering broadcastm intrinsic - clang part

2017-10-09 Thread jina via Phabricator via cfe-commits
jina.nahias created this revision. https://reviews.llvm.org/D38683 Files: include/clang/Basic/BuiltinsX86.def lib/Headers/avx512cdintrin.h lib/Headers/avx512vlcdintrin.h test/CodeGen/avx512cdintrin.c test/CodeGen/avx512vlcd-builtins.c Index: test/CodeGen/avx512vlcd-builtins.c =

[PATCH] D38627: [clangd] Added move-only function helpers.

2017-10-09 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315210: [clangd] Added move-only function helpers. (authored by ibiryukov). Repository: rL LLVM https://reviews.llvm.org/D38627 Files: clang-tools-extra/trunk/clangd/ClangdServer.cpp clang-tools-e

[PATCH] D38617: Set PreprocessorOpts.GeneratePreamble=true in PrecompiledPreamble.

2017-10-09 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315212: Set PreprocessorOpts.GeneratePreamble=true in PrecompiledPreamble. (authored by ibiryukov). Repository: rL LLVM https://reviews.llvm.org/D38617 Files: cfe/trunk/lib/Frontend/ASTUnit.cpp cf

[PATCH] D37970: [clangd] Added a command-line arg to mirror clangd input into a file.

2017-10-09 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315214: [clangd] Added a command-line arg to mirror clangd input into a file. (authored by ibiryukov). Changed prior to commit: https://reviews.llvm.org/D37970?vs=115628&id=118223#toc Repository: rL

[PATCH] D38643: PR13575: Fix USR mangling for fixed-size arrays.

2017-10-09 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315236: PR13575: Fix USR mangling for fixed-size arrays (authored by jkorous). Changed prior to commit: https://reviews.llvm.org/D38643?vs=118247&id=118257#toc Repository: rL LLVM https://reviews.ll

[PATCH] D38711: typos in documentation?

2017-10-09 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315252: Fix typos in documentation (authored by jkorous). Changed prior to commit: https://reviews.llvm.org/D38711?vs=118295&id=118296#toc Repository: rL LLVM https://reviews.llvm.org/D38711 Files:

[PATCH] D38707: PR13575: Fix USR mangling for functions taking function pointers as arguments.

2017-10-09 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315255: R13575: Fix USR mangling for function pointer types (authored by jkorous). Changed prior to commit: https://reviews.llvm.org/D38707?vs=118287&id=118300#toc Repository: rL LLVM https://review

[PATCH] D37478: [analyzer] Implement pointer arithmetic on constants

2017-10-10 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315296: [analyzer] Implement pointer arithmetic on constants (authored by xazax). Changed prior to commit: https://reviews.llvm.org/D37478?vs=118182&id=118342#toc Repository: rL LLVM https://reviews

[PATCH] D37805: [analyzer] PthreadLock: add printState().

2017-10-10 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315298: [analyzer] PthreadLockChecker: Add printState() method for self-debugging. (authored by dergachev). Changed prior to commit: https://reviews.llvm.org/D37805?vs=115028&id=118349#toc Repository:

[PATCH] D31541: [analyzer] MisusedMovedObjectChecker: Add a printState() method.

2017-10-10 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315300: [analyzer] MisusedMovedObject: Add printState() method for self-debugging. (authored by dergachev). Changed prior to commit: https://reviews.llvm.org/D31541?vs=93661&id=118350#toc Repository:

[PATCH] D31538: [analyzer] MisusedMovedObjectChecker: Fix a false positive on state-resetting a base-class sub-object.

2017-10-10 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315301: [analyzer] MisusedMovedObject: Fix state-resetting a base-class sub-object. (authored by dergachev). Changed prior to commit: https://reviews.llvm.org/D31538?vs=93652&id=118351#toc Repository:

[PATCH] D38755: Fixed crash during indexing default template template param

2017-10-10 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315367: Fix indexer crash for default template template parameter value (authored by jkorous). Changed prior to commit: https://reviews.llvm.org/D38755?vs=118451&id=118476#toc Repository: rL LLVM ht

[PATCH] D38473: Include getting generated struct offsets in CodegenABITypes

2017-10-10 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315392: Include getting generated struct offsets in CodegenABITypes (authored by adrian). Changed prior to commit: https://reviews.llvm.org/D38473?vs=117806&id=118505#toc Repository: rL LLVM https:/

[PATCH] D38781: [X86] Add CLWB intrinsic. clang part

2017-10-12 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315607: [X86] Add CLWB intrinsic. clang part (authored by ctopper). Changed prior to commit: https://reviews.llvm.org/D38781?vs=118816&id=118822#toc Repository: rL LLVM https://reviews.llvm.org/D387

[PATCH] D38101: [Sema] Diagnose tautological comparison with type's min/max values

2017-10-12 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315614: [Sema] Diagnose tautological comparison with type's min/max values (authored by lebedevri). Changed prior to commit: https://reviews.llvm.org/D38101?vs=118833&id=118835#toc Repository: rL LLV

[PATCH] D38813: [X86] Add skeleton support for "knm" cpu - clang side

2017-10-13 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315723: [X86] Add skeleton support for knm cpu (authored by ctopper). Changed prior to commit: https://reviews.llvm.org/D38813?vs=118650&id=118945#toc Repository: rL LLVM https://reviews.llvm.org/D3

[PATCH] D38877: [analyzer] RetainCount: Accept "safe" CFRetain wrappers.

2017-10-13 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315736: [analyzer] RetainCount: Ignore annotations on user-made CFRetain wrappers. (authored by dergachev). Changed prior to commit: https://reviews.llvm.org/D38877?vs=118891&id=118953#toc Repository:

[PATCH] D38797: [analyzer] CStringChecker: pr34460: Admit that some casts are hard to model.

2017-10-13 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315742: [analyzer] CStringChecker: pr34460: Avoid a crash when a cast is not modeled. (authored by dergachev). Changed prior to commit: https://reviews.llvm.org/D38797?vs=118597&id=118964#toc Repositor

[PATCH] D23963: [analyzer] pr28449 - Move literal rvalue construction away from RegionStore.

2017-10-13 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315750: [analyzer] pr28449: Fix support for various array initializers. (authored by dergachev). Changed prior to commit: https://reviews.llvm.org/D23963?vs=118625&id=118974#toc Repository: rL LLVM

[PATCH] D38913: [ubsan] Don't emit function signatures for virtual methods

2017-10-13 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL315786: [ubsan] Don't emit function signatures for non-static member functions (authored by vedantk). Changed prior to commit: https://reviews.llvm.org/D38913?vs=118999&id=119004#toc Repository: rL L

<    2   3   4   5   6   7   8   9   10   11   >