[PATCH] D41523: xmmintrin.h documentation fixes and updates

kromanova added a comment. In https://reviews.llvm.org/D41523#968776, @craig.topper wrote: > The builtins are tested in tests like test/CodeGen/sse-builtins.c, Thank you! I wonder if -Wdocumentation is working... I have enabled it for a few tests, like avx-builtins.c, sse-builtins.c and re-r

[PATCH] D41842: [libunwind][MIPS]: Rename Registers_mips_n64 to Registers_mips_newabi.

compnerd accepted this revision. compnerd added a comment. This revision is now accepted and ready to land. Thanks for splitting this up! https://reviews.llvm.org/D41842 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/c

Re: r321855 - Reapply r321781: [Modules] Allow modules specified by -fmodule-map-file to shadow implicitly found ones

I haven't done a ton of investigation yet, but I suspect this commit is responsible for breaking my LLVM module selfhost build, which uses libc++ as the STL, and builds LLD (No bots appear to test this configuration). The error message can be found here: https://gist.github.com/EricWF/fb5e1d18eb10

[PATCH] D41516: emmintrin.h documentation fixes and updates

kromanova added inline comments. Comment at: cfe/trunk/lib/Headers/emmintrin.h:4683 /// -/// This intrinsic has no corresponding instruction. +/// This intrinsic corresponds to the MOVDQ2Q instruction. /// efriedma wrote: > kromanova wrote: > > kromanova wrot

[PATCH] D41829: [cmake] Add cache file to bootstrap 2-stage linux cross compile on Darwin.

hintonda updated this revision to Diff 129028. hintonda added a comment. - Also pass LLVM_CONFIG_EXE. Repository: rC Clang https://reviews.llvm.org/D41829 Files: cmake/caches/Linux.cmake Index: cmake/caches/Linux.cmake === -

[PATCH] D41523: xmmintrin.h documentation fixes and updates

dyung updated this revision to Diff 129021. dyung added a comment. Update review based on feedback. https://reviews.llvm.org/D41523 Files: lib/Headers/xmmintrin.h Index: lib/Headers/xmmintrin.h === --- lib/Headers/xmmintrin.h ++

[PATCH] D41516: emmintrin.h documentation fixes and updates

efriedma added inline comments. Comment at: cfe/trunk/lib/Headers/emmintrin.h:4683 /// -/// This intrinsic has no corresponding instruction. +/// This intrinsic corresponds to the MOVDQ2Q instruction. /// kromanova wrote: > kromanova wrote: > > I'm not sure a

[PATCH] D41516: emmintrin.h documentation fixes and updates

kromanova added inline comments. Herald added a subscriber: llvm-commits. Comment at: cfe/trunk/lib/Headers/emmintrin.h:3865 /// -/// This intrinsic corresponds to the VPUNPCKLQDQ / PUNPCKLQDQ -/// instruction. +/// This intrinsic does not correspond to a specific instructio

[PATCH] D41266: [analyzer] With c++-allocator-inlining, fix memory space for operator new pointers.

dcoughlin added inline comments. Comment at: lib/StaticAnalyzer/Core/ExprEngineCXX.cpp:477 +bool ExprEngine::isStandardGlobalOperatorNewFunction(const CXXNewExpr *CNE) { + const FunctionDecl *FD = CNE->getOperatorNew(); + if (FD && !isa(FD) && !FD->isVariadic()) { -

[PATCH] D41575: [index] Return when DC is null in handleReference

MaskRay added a comment. In https://reviews.llvm.org/D41575#970412, @akyrtzi wrote: > Ah, sorry I mislead you. To test this try using `c-index-test -index-file > /path/to/file`, see other examples in `test/Index`, e.g. > `test/Index/index-file.cpp` Thanks for the command. It is really helpful

[PATCH] D41800: [analyzer] Use a custom program point for the check::NewAllocator callback.

dcoughlin accepted this revision. dcoughlin added inline comments. This revision is now accepted and ready to land. Comment at: lib/StaticAnalyzer/Core/CheckerManager.cpp:491 NodeBuilder &Bldr, ExplodedNode *Pred) { // TODO: Does this deserve a custom

[PATCH] D41408: [analyzer] NFC: Fix nothrow operator new definition in a test.

dcoughlin added a comment. LGTM as well. Repository: rC Clang https://reviews.llvm.org/D41408 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D41406: [analyzer] Add a new checker callback, check::NewAllocator.

dcoughlin accepted this revision. dcoughlin added a comment. This revision is now accepted and ready to land. Looks good to me with some minor nits inside (and also a request to consider factoring out common code). Comment at: include/clang/StaticAnalyzer/Core/CheckerManager.h

[PATCH] D41834: [Lex] Fix handling numerical literals ending with ' and signed exponent.

vsapsai planned changes to this revision. vsapsai added a comment. Yep, the plan sounds reasonable. I also noticed that we have if (*s == '+' || *s == '-') s++; // sign code in `NumericLiteralParser::ParseNumberStartingWithZero` too. I plan to make the same change for hexadecimal numbers and

[PATCH] D41517: mmintrin.h documentation fixes and updates

efriedma added inline comments. Comment at: lib/Headers/mmintrin.h:55 /// -/// This intrinsic corresponds to the VMOVD / MOVD instruction. +/// This intrinsic corresponds to the MOVD instruction. /// craig.topper wrote: > kromanova wrote: > > I tried clang

[PATCH] D41845: [libcxx] clean up and complete

timshen updated this revision to Diff 129017. timshen added a comment. Update on template variable #ifdefs in tests. https://reviews.llvm.org/D41845 Files: libcxx/include/experimental/simd libcxx/test/std/experimental/simd/simd.cons/load.pass.cpp libcxx/test/std/experimental/simd/simd.mas

[PATCH] D41834: [Lex] Fix handling numerical literals ending with ' and signed exponent.

rsmith added a comment. The lexer is doing the right thing; per the C++ lexical rules, the `+1` is not part of the token in this case. I don't think this fix is in the right place; we will still examine characters after the end of the literal, even with this applied, and that doesn't seem righ

[PATCH] D41517: mmintrin.h documentation fixes and updates

dyung updated this revision to Diff 129015. dyung added a comment. Updating diff based on review feedback. https://reviews.llvm.org/D41517 Files: lib/Headers/mmintrin.h Index: lib/Headers/mmintrin.h === --- lib/Headers/mmintrin

r322052 - Fix use-after-free found by address-san on -r322028.

Author: erichkeane Date: Mon Jan 8 17:09:12 2018 New Revision: 322052 URL: http://llvm.org/viewvc/llvm-project?rev=322052&view=rev Log: Fix use-after-free found by address-san on -r322028. r322028 attempted to remove something from the "Manglings" list when it was no longer valid, and did so wit

[PATCH] D41756: [libcxx] implement simd_mask<> casts and some horizontal operations.

timshen updated this revision to Diff 129013. timshen added a comment. Rebased. https://reviews.llvm.org/D41756 Files: libcxx/include/experimental/simd libcxx/test/std/experimental/simd/simd.casts/to_compatible.pass.cpp libcxx/test/std/experimental/simd/simd.casts/to_fixed_size.pass.cpp

[PATCH] D41749: [analyzer] suppress nullability inference from a macro when result is used in another macro

dcoughlin accepted this revision. dcoughlin added a comment. Yes, this looks great! https://reviews.llvm.org/D41749 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D41747: [libcxx] implement simd_mask<> and operators.

timshen updated this revision to Diff 129012. timshen added a comment. Rebased. https://reviews.llvm.org/D41747 Files: libcxx/include/experimental/simd libcxx/test/std/experimental/simd/simd.elementwise/operators.pass.cpp libcxx/test/std/experimental/simd/simd.mask.access/default.pass.cpp

[PATCH] D41422: [libcxx] implement operators and reduction.

timshen updated this revision to Diff 129011. timshen added a comment. Rebased. https://reviews.llvm.org/D41422 Files: libcxx/include/experimental/simd libcxx/test/std/experimental/simd/simd.elementwise/clamp.pass.cpp libcxx/test/std/experimental/simd/simd.elementwise/max.pass.cpp libcx

[PATCH] D41148: [libcxx] implement declarations based on P0214R7.

timshen updated this revision to Diff 129007. timshen added a comment. Rebased. https://reviews.llvm.org/D41148 Files: libcxx/include/experimental/__config libcxx/include/experimental/simd libcxx/test/std/experimental/simd/nothing_to_do.pass.cpp libcxx/test/std/experimental/simd/simd.ca

[PATCH] D41415: [libcxx] implement casts.

timshen updated this revision to Diff 129010. timshen added a comment. Rebased. https://reviews.llvm.org/D41415 Files: libcxx/include/experimental/simd libcxx/test/std/experimental/simd/simd.casts/simd_cast.pass.cpp libcxx/test/std/experimental/simd/simd.casts/static_simd_cast.pass.cpp

[PATCH] D41412: [libcxx] implement concat() and split()

timshen updated this revision to Diff 129009. timshen added a comment. Rebased. https://reviews.llvm.org/D41412 Files: libcxx/include/experimental/simd libcxx/test/std/experimental/simd/simd.horizontal/concat.pass.cpp libcxx/test/std/experimental/simd/simd.horizontal/split.pass.cpp Index

[PATCH] D41376: [libcxx] Implement ABI for Clang/GCC vector extension, constructors, copy_from and copy_to.

timshen updated this revision to Diff 129008. timshen added a comment. Rebased. https://reviews.llvm.org/D41376 Files: libcxx/include/__config libcxx/include/experimental/__config libcxx/include/experimental/simd libcxx/include/utility libcxx/test/std/experimental/simd/simd.abi/vector

[PATCH] D41845: [libcxx] clean up and complete

timshen updated this revision to Diff 129006. timshen added a comment. Implement memory_alignment. https://reviews.llvm.org/D41845 Files: libcxx/include/experimental/simd Index: libcxx/include/experimental/simd === --- libcxx/in

[PATCH] D41845: [libcxx] clean up and complete

timshen created this revision. timshen added reviewers: mclow.lists, EricWF. Herald added a subscriber: sanjoy. The cleanup patch adds sevreal TODOs for the following unimplemented features: *) aligned load *) simd<> version of functions This patch declares the completion of my implementation of

[PATCH] D41844: [libcxx] implement mask reductions

timshen created this revision. timshen added reviewers: mclow.lists, EricWF. Herald added a subscriber: sanjoy. This is not efficiently implemented typename V::value_type reduce(const const_where_expression &x, typename V::value_type neutral_element, BinaryOperation binary_op); as w

[PATCH] D41843: [libcxx] implement where expressions.

timshen created this revision. timshen added reviewers: mclow.lists, EricWF. Herald added a subscriber: sanjoy. Where expressions have three cases: *) const_where_expression, simd<...>>, we store two references to the mask and the simd<> value. *) const_where_expression, simd_mask<...>>, we store

[PATCH] D41517: mmintrin.h documentation fixes and updates

craig.topper added inline comments. Comment at: lib/Headers/mmintrin.h:1402 /// -/// This intrinsic corresponds to the VPSHUFLW / PSHUFLW instruction. +/// This intrinsic corresponds to the PSHUFLW instruction. /// dyung wrote: > craig.topper wrote: > > Thi

[PATCH] D41517: mmintrin.h documentation fixes and updates

dyung added inline comments. Comment at: lib/Headers/mmintrin.h:1402 /// -/// This intrinsic corresponds to the VPSHUFLW / PSHUFLW instruction. +/// This intrinsic corresponds to the PSHUFLW instruction. /// craig.topper wrote: > This is overly specific Jus

[PATCH] D41523: xmmintrin.h documentation fixes and updates

dyung added inline comments. Comment at: lib/Headers/xmmintrin.h:1927 /// -/// This intrinsic corresponds to the VPEXTRQ / MOVQ instruction. +/// This intrinsic corresponds to the VPEXTRQ / PEXTRQ instruction. /// RKSimon wrote: > Not necessarily, it could

[PATCH] D41788: [DeclPrinter] Fix two cases that crash clang -ast-print.

jlebar added a comment. I strongly approve of fixing these crashes, but I don't think I can say with confidence whether this change is correct. https://reviews.llvm.org/D41788 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llv

[PATCH] D41842: [libunwind][MIPS]: Rename Registers_mips_n64 to Registers_mips_newabi.

bsdjhb created this revision. bsdjhb added reviewers: compnerd, sdardis. Herald added a subscriber: arichardson. This is in preparation for adding support for N32 unwinding which reuses the newabi register class. https://reviews.llvm.org/D41842 Files: include/__libunwind_config.h src/Regist

[PATCH] D33563: Track whether a unary operation can overflow

efriedma accepted this revision. efriedma added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D33563 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/c

[PATCH] D41841: [OpenMP] Fix handling of clause on wrong directive

jdenny created this revision. jdenny added a reviewer: ABataev. First, this patch fixes an assert failure when, for example, "omp for" has num_teams. Second, this patch prevents duplicate diagnostics when, for example, "omp for" has uniform. This patch makes the general assumption (even where it

[PATCH] D41575: [index] Return when DC is null in handleReference

akyrtzi added a comment. Ah, sorry I mislead you. To test this try using `c-index-test -index-file /path/to/file`, see other examples in `test/Index`, e.g. `test/Index/index-file.cpp` Repository: rL LLVM https://reviews.llvm.org/D41575 ___ cfe-

[PATCH] D40513: [libcxx] Support the use of compiler-rt in lit tests

This revision was automatically updated to reflect the committed changes. Closed by commit rCXX322044: [libcxx] Support the use of compiler-rt in lit tests (authored by phosek, committed by ). Repository: rCXX libc++ https://reviews.llvm.org/D40513 Files: test/CMakeLists.txt test/lit.site

[libcxx] r322044 - [libcxx] Support the use of compiler-rt in lit tests

Author: phosek Date: Mon Jan 8 15:36:53 2018 New Revision: 322044 URL: http://llvm.org/viewvc/llvm-project?rev=322044&view=rev Log: [libcxx] Support the use of compiler-rt in lit tests Don't link tests against libgcc when compiler-rt is being used. Differential Revision: https://reviews.llvm.or

[PATCH] D41837: Add Function multiversion to the release notes.

This revision was automatically updated to reflect the committed changes. Closed by commit rC322043: Document attribute target multiversioning. (authored by erichkeane, committed by ). Changed prior to commit: https://reviews.llvm.org/D41837?vs=128982&id=128995#toc Repository: rC Clang http

r322043 - Document attribute target multiversioning.

Author: erichkeane Date: Mon Jan 8 15:36:29 2018 New Revision: 322043 URL: http://llvm.org/viewvc/llvm-project?rev=322043&view=rev Log: Document attribute target multiversioning. Add attribute target multiversioning to the release notes. Additionally adds multiversioning support to the attribut

[PATCH] D41837: Add Function multiversion to the release notes.

rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. Looks fine. Can you also update include/clang/Basic/AttrDocs.td to mention multiversioning in `TargetDocs`? Then you can add a few words here to say "consult the documentation for the target a

[PATCH] D40513: [libcxx] Support the use of compiler-rt in lit tests

EricWF accepted this revision. EricWF added a comment. This revision is now accepted and ready to land. Herald added a subscriber: llvm-commits. LGTM. Honestly, I'm surprised we don't already have a way to configure like this. Repository: rL LLVM https://reviews.llvm.org/D40513 __

[PATCH] D33563: Track whether a unary operation can overflow

rsmith added a comment. Looks fine to me, wait to see if Eli has more comments. https://reviews.llvm.org/D33563 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D41368: [libc++] Ignore bogus tautologic comparison warnings

smeenai abandoned this revision. smeenai added a comment. The warning was removed from `-Wall`. Repository: rCXX libc++ https://reviews.llvm.org/D41368 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/

[libcxx] r322039 - Revert "Make rehash(0) work with ubsan's unsigned-integer-overflow."

Author: danalbert Date: Mon Jan 8 14:57:12 2018 New Revision: 322039 URL: http://llvm.org/viewvc/llvm-project?rev=322039&view=rev Log: Revert "Make rehash(0) work with ubsan's unsigned-integer-overflow." Seems to have broken some tests since I first wrote this a while back. Will reland after che

[PATCH] D41409: [analyzer] Fix intermediate diagnostics on paths that go through operator new().

dcoughlin added a comment. LGTM as well. Repository: rC Clang https://reviews.llvm.org/D41409 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r322038 - [X86] Replace cvt*2mask intrinsics with native IR using 'icmp slt X, zeroinitializer.

Author: ctopper Date: Mon Jan 8 14:37:56 2018 New Revision: 322038 URL: http://llvm.org/viewvc/llvm-project?rev=322038&view=rev Log: [X86] Replace cvt*2mask intrinsics with native IR using 'icmp slt X, zeroinitializer. Modified: cfe/trunk/lib/CodeGen/CGBuiltin.cpp cfe/trunk/test/CodeGen

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

malaperle added a comment. @ilya-biryukov Hi! I'll be updating William's patches that were in progress. I just have a few comments/question before I send a new update. (I also don't know if I can update this diff or I have to create a new diff on Phabricator... I guess we'll see!!).

[PATCH] D33563: Track whether a unary operation can overflow

aaron.ballman updated this revision to Diff 128984. aaron.ballman marked 2 inline comments as done. aaron.ballman added a comment. Updated based on review feedback. https://reviews.llvm.org/D33563 Files: include/clang/AST/Expr.h lib/AST/ASTDumper.cpp lib/AST/ASTImporter.cpp lib/AST/Expr

[PATCH] D33563: Track whether a unary operation can overflow

aaron.ballman added inline comments. Comment at: test/Misc/ast-dump-stmt.c:66 + // CHECK:ImplicitCastExpr + // CHECK: DeclRefExpr{{.*}}'T2' 'int' +} efriedma wrote: > aaron.ballman wrote: > > efriedma wrote: > > > What does it mean for bitwise

[libcxx] r322034 - Apparently 'C++14' is different than 'c++14'

Author: marshall Date: Mon Jan 8 14:16:30 2018 New Revision: 322034 URL: http://llvm.org/viewvc/llvm-project?rev=322034&view=rev Log: Apparently 'C++14' is different than 'c++14' Modified: libcxx/trunk/test/std/utilities/function.objects/func.search/func.search.bm/hash.pass.cpp Modified:

[PATCH] D41837: Add Function multiversion to the release notes.

erichkeane updated this revision to Diff 128982. erichkeane added a comment. Ah, right :) Slipped my mind that ELF != x86-linux. https://reviews.llvm.org/D41837 Files: docs/ReleaseNotes.rst Index: docs/ReleaseNotes.rst === ---

[PATCH] D41837: Add Function multiversion to the release notes.

echristo added a comment. I think you're missing that right now it's x86 only yes? :) -eric Repository: rC Clang https://reviews.llvm.org/D41837 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listin

[PATCH] D41837: Add Function multiversion to the release notes.

erichkeane created this revision. erichkeane added reviewers: rsmith, hans, aaron.ballman, echristo. Richard suggested that I add this feature to the release notes, so I was hoping someone (anyone willing:) ) could do a quick read through for me. Thanks! -Erich Repository: rC Clang https://r

[PATCH] D40743: Make rehash(0) work with ubsan's unsigned-integer-overflow.

danalbert added inline comments. Comment at: include/__hash_table:2141 __n = 2; else if (__n & (__n - 1)) __n = __next_prime(__n); mclow.lists wrote: > danalbert wrote: > > With `rehash(0)` this is `0 & (0 - 1)`, which triggers > > unsigne

[PATCH] D40743: Make rehash(0) work with ubsan's unsigned-integer-overflow.

This revision was automatically updated to reflect the committed changes. Closed by commit rCXX322031: Make rehash(0) work with ubsan's unsigned-integer-overflow. (authored by danalbert, committed by ). Changed prior to commit: https://reviews.llvm.org/D40743?vs=125193&id=128979#toc Repository

[libcxx] r322031 - Make rehash(0) work with ubsan's unsigned-integer-overflow.

Author: danalbert Date: Mon Jan 8 13:49:12 2018 New Revision: 322031 URL: http://llvm.org/viewvc/llvm-project?rev=322031&view=rev Log: Make rehash(0) work with ubsan's unsigned-integer-overflow. Reviewers: mclow.lists, EricWF Reviewed By: mclow.lists Subscribers: cfe-commits Differential Revi

r322030 - PR35862: Suppress -Wmissing-variable-declarations warning on inline variables,

Author: rsmith Date: Mon Jan 8 13:46:42 2018 New Revision: 322030 URL: http://llvm.org/viewvc/llvm-project?rev=322030&view=rev Log: PR35862: Suppress -Wmissing-variable-declarations warning on inline variables, variable templates, and instantiations thereof. Modified: cfe/trunk/lib/Sema/Sema

[PATCH] D41706: [Driver] Update default sanitizer blacklist location

compnerd accepted this revision. compnerd added a comment. This revision is now accepted and ready to land. This seems fine, but will need https://reviews.llvm.org/D41673 to go in at the same time. The sanitizers are pretty tightly coupled with the compiler, so I don't think that it is too big

[PATCH] D41039: Add support for attribute "trivial_abi"

rsmith added a comment. I'd like to see more testing for the template instantiation case. I don't see any test coverage for the "attribute only affects instantiations whose members are trivial-for-calls" part. Comment at: include/clang/Sema/Sema.h:2239 bool SpecialMemberIs

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

erichkeane closed this revision. erichkeane marked 2 inline comments as done. erichkeane added a comment. Closed with revision 322028 https://reviews.llvm.org/D40819 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-b

r322028 - Implement Attribute Target MultiVersioning

Author: erichkeane Date: Mon Jan 8 13:34:17 2018 New Revision: 322028 URL: http://llvm.org/viewvc/llvm-project?rev=322028&view=rev Log: Implement Attribute Target MultiVersioning GCC's attribute 'target', in addition to being an optimization hint, also allows function multiversioning. We current

r322027 - [DOXYGEN] Fix doxygen and content issues in avxintrin.h

Author: dyung Date: Mon Jan 8 13:21:17 2018 New Revision: 322027 URL: http://llvm.org/viewvc/llvm-project?rev=322027&view=rev Log: [DOXYGEN] Fix doxygen and content issues in avxintrin.h - Fix incorrect wording in various intrinsic descriptions. Previously the descriptions used "low-order" and

r322024 - Factor out comparison handling for arithmetic types.

Author: rsmith Date: Mon Jan 8 13:12:04 2018 New Revision: 322024 URL: http://llvm.org/viewvc/llvm-project?rev=322024&view=rev Log: Factor out comparison handling for arithmetic types. This is not quite NFC: we don't perform the usual arithmetic conversions unless we have an operand of arithmeti

[PATCH] D41834: [Lex] Fix handling numerical literals ending with ' and signed exponent.

vsapsai added a comment. This fixes the OSS-Fuzz bug but I don't know if it is sufficient. Should I also make `Lexer::LexNumericConstant` to include `+1` part as tok::numeric_constant? https://reviews.llvm.org/D41834 ___ cfe-commits mailing list cf

[PATCH] D41834: [Lex] Fix handling numerical literals ending with ' and signed exponent.

vsapsai created this revision. vsapsai added reviewers: rsmith, t.p.northover. For input `0'e+1` lexer tokenized as numeric constant only `0'e`. Later NumericLiteralParser skipped 0 and ' as digits and parsed `e+1` as valid exponent going past the end of the token. Because it didn't mark numeric l

[PATCH] D41039: Add support for attribute "trivial_abi"

rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. Thanks, looks good to me. https://reviews.llvm.org/D41039 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-b

[PATCH] D41271: [RISCV] Propagate -mabi and -march values to GNU assembler.

apazos added a comment. This is ready to merge, just waiting for the dependence https://reviews.llvm.org/D39963 to be merged first. https://reviews.llvm.org/D41271 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bi

[PATCH] D40023: [RISCV] Implement ABI lowering

apazos added a comment. Hi Alex, just a reminder, it looks like Eli's and David's comments have not been addressed yet. https://reviews.llvm.org/D40023 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/li

[PATCH] D41713: [Myriad] Remove invalidated -elf flag for MoviAsm

This revision was automatically updated to reflect the committed changes. Closed by commit rC322023: [Myriad] Remove invalidated -elf flag for MoviAsm (authored by waltl, committed by ). Changed prior to commit: https://reviews.llvm.org/D41713?vs=128557&id=128970#toc Repository: rC Clang ht

r322023 - [Myriad] Remove invalidated -elf flag for MoviAsm

Author: waltl Date: Mon Jan 8 12:36:08 2018 New Revision: 322023 URL: http://llvm.org/viewvc/llvm-project?rev=322023&view=rev Log: [Myriad] Remove invalidated -elf flag for MoviAsm Summary: The flag has been deprecated, and is becoming invalid in the latest MDK. Reviewers: jyknight Subscribers

[PATCH] D39963: [RISCV] Add initial RISC-V target and driver support

apazos accepted this revision. apazos added a comment. This revision is now accepted and ready to land. Please merge this patch, it looks in good shape. This patch is required for any RISCV build. https://reviews.llvm.org/D39963 ___ cfe-commits mai

[PATCH] D41507: avxintrin.h documentation fixes and updates

kromanova accepted this revision. kromanova added a comment. LGTM too. https://reviews.llvm.org/D41507 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r322022 - [OPENMP] Fix debug info for outlined functions in NVPTX + add more tests.

Author: abataev Date: Mon Jan 8 12:09:47 2018 New Revision: 322022 URL: http://llvm.org/viewvc/llvm-project?rev=322022&view=rev Log: [OPENMP] Fix debug info for outlined functions in NVPTX + add more tests. Fixed name of emitted outlined functions in NVPTX target + extra tests for the debug inf

[PATCH] D33563: Track whether a unary operation can overflow

efriedma added inline comments. Comment at: test/Misc/ast-dump-stmt.c:66 + // CHECK:ImplicitCastExpr + // CHECK: DeclRefExpr{{.*}}'T2' 'int' +} aaron.ballman wrote: > efriedma wrote: > > What does it mean for bitwise complement to "overflow"? >

[PATCH] D41830: [libc++] Fix PR#35780 - make std::experimental::filesystem::remove return false if the file doesn't exist

TyanNN created this revision. TyanNN added a reviewer: EricWF. Herald added a subscriber: cfe-commits. Previously it thrown an error if the file didn't exist. PR#35780 Repository: rCXX libc++ https://reviews.llvm.org/D41830 Files: src/experimental/filesystem/operations.cpp test/libcxx/

[libcxx] r322021 - Change add_ten to add_one to avoid triggering ubsan integer overflow.

Author: bion Date: Mon Jan 8 11:45:16 2018 New Revision: 322021 URL: http://llvm.org/viewvc/llvm-project?rev=322021&view=rev Log: Change add_ten to add_one to avoid triggering ubsan integer overflow. Modified: libcxx/trunk/test/std/numerics/numeric.ops/transform.exclusive.scan/transform_exc

[PATCH] D41829: [cmake] Add cache file to bootstrap 2-stage linux cross compile on Darwin.

hintonda created this revision. hintonda added reviewers: beanz, compnerd, phosek, smeenai. Herald added a subscriber: mgorny. Add cache file to bootstrap 2-stage linux cross compile on Darwin. Repository: rC Clang https://reviews.llvm.org/D41829 Files: cmake/caches/Linux.cmake Index: cm

[PATCH] D41039: Add support for attribute "trivial_abi"

ahatanak updated this revision to Diff 128960. ahatanak marked 7 inline comments as done. ahatanak added a comment. Address review comments. https://reviews.llvm.org/D41039 Files: include/clang/AST/Decl.h include/clang/AST/DeclCXX.h include/clang/AST/Type.h include/clang/Basic/Attr.td

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

rsmith accepted this revision. rsmith added a comment. Looks good to me with just a few more tweaks (assuming these comments don't uncover any new issues). Thank you! Comment at: lib/AST/ASTContext.cpp:9490 + assert(FD->isMultiVersion() && "Only valid for multiversioned funct

[libcxx] r322019 - Add the C++17 extensions to std::search. Include the default searcher, but not the Boyer-Moore or Boyer-Moore-Horspool searcher (yet). BUT put the BM and BMH tests in place, marked

Author: marshall Date: Mon Jan 8 11:18:00 2018 New Revision: 322019 URL: http://llvm.org/viewvc/llvm-project?rev=322019&view=rev Log: Add the C++17 extensions to std::search. Include the default searcher, but not the Boyer-Moore or Boyer-Moore-Horspool searcher (yet). BUT put the BM and BMH tes

Re: r322018 - [OPENMP] Current status of OpenMP support.

Can we backport this page to release_60? I think the documented support is also valid for 6.0 or did I miss recent commits that added support for new directives / clauses? Am 2018-01-08 20:02, schrieb Alexey Bataev via cfe-commits: Author: abataev Date: Mon Jan 8 11:02:51 2018 New Revision: 3

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

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 rC322018: [OPENMP] Current status of OpenMP support. (authored by ABataev, committed by ). Changed prior to commit: https

r322018 - [OPENMP] Current status of OpenMP support.

Author: abataev Date: Mon Jan 8 11:02:51 2018 New Revision: 322018 URL: http://llvm.org/viewvc/llvm-project?rev=322018&view=rev Log: [OPENMP] Current status of OpenMP support. Summary: Some info about supported features of OpenMP 4.5-5.0. Reviewers: hfinkel, rsmith Subscribers: kkwli0, Hahnfel

[PATCH] D41575: [index] Return when DC is null in handleReference

This revision was automatically updated to reflect the committed changes. Closed by commit rL322017: [index] Return when DC is null in handleReference (authored by MaskRay, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/D41575 Files: cf

[PATCH] D41655: [clang-tidy] New check bugprone-unused-return-value

khuttun added inline comments. Comment at: test/clang-tidy/bugprone-unused-return-value.cpp:163 + +void noWarning() { + auto AsyncRetval1 = std::async(increment, 42); aaron.ballman wrote: > khuttun wrote: > > aaron.ballman wrote: > > > khuttun wrote: > > > > aar

[PATCH] D41575: [index] Return when DC is null in handleReference

MaskRay updated this revision to Diff 128953. MaskRay added a comment. rebase Repository: rC Clang https://reviews.llvm.org/D41575 Files: tools/libclang/CXIndexDataConsumer.cpp Index: tools/libclang/CXIndexDataConsumer.cpp =

[PATCH] D41575: [index] Return when DC is null in handleReference

MaskRay added a comment. @akyrtzi When I run `c-index-test core -print-source-symbols -- a.cc` on template struct actor; template class Actor = actor> struct terminal; the issue disappears. It emerges only when `clang_indexTranslationUnit` is called with interactions of other thing

[PATCH] D41539: [CodeGen] Decorate aggregate accesses with TBAA tags

kosarev added a comment. OK, I'm reading your response so that this patch may significantly increase the number of cases where we propagate TBAA tags from memory-transfer intrinsic calls, which means potentially more cases where ignoring may_alias would lead to problems. If so, then I tend to a

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

erichkeane updated this revision to Diff 128950. erichkeane marked 2 inline comments as done. erichkeane added a comment. All of @rsmith 's comments on the last patch. 1 "open" item on the duplicate discover in the resolver generation, but hopefully this otherwise acceptable. https://reviews.l

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

erichkeane marked 7 inline comments as done. erichkeane added a comment. patch incoming! Comment at: include/clang/AST/ASTContext.h:2643-2648 +for (auto *CurDecl : + FD->getDeclContext()->getRedeclContext()->lookup(FD->getDeclName())) { + FunctionDecl *CurFD =

[PATCH] D41575: [index] Return when DC is null in handleReference

akyrtzi added a comment. Could you add a test case for this change ? See examples in `test/Index/Core`. Also for the test case code: `template class Actor = actor>`, is the `actor` reference in this code reported ? See the test examples on how to print out and test how the source symbols are in

[PATCH] D41575: [index] Return when DC is null in handleReference

sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. Sorry, I was hoping @akyrtzi would see this. I don't know enough about the code to know whether DC==null is an indicator of some other problem, but if this is crashing today then let's no

[libcxx] r322011 - Document upcoming TS feature removal

Author: marshall Date: Mon Jan 8 09:43:46 2018 New Revision: 322011 URL: http://llvm.org/viewvc/llvm-project?rev=322011&view=rev Log: Document upcoming TS feature removal Added: libcxx/trunk/www/TS_deprecation.html Added: libcxx/trunk/www/TS_deprecation.html URL: http://llvm.org/viewvc/llv

Re: r321997 - Avoid assumption that lit tests are writable. NFC

Hi David (and the list this time!), If X is readonly, then after `cp X Y`, Y is also readonly. The `cat` version doesn't propagate permissions. The environment for lit tests isn't really spelled out, but relying on the input files being +w doesn't seem obviously reasonable. Google's internal runn

Re: r321816 - [OPENMP] Add debug info for generated functions.

Will add some more tests later today - Best regards, Alexey Bataev 08.01.2018 11:13, David Blaikie пишет: > Rough guess: That seems like a lot of code changes for relatively > little test changes - are all parts of this change tested? (Might well > be - just lots of layers to plumb thi

[PATCH] D41655: [clang-tidy] New check bugprone-unused-return-value

aaron.ballman added inline comments. Comment at: test/clang-tidy/bugprone-unused-return-value.cpp:163 + +void noWarning() { + auto AsyncRetval1 = std::async(increment, 42); khuttun wrote: > aaron.ballman wrote: > > khuttun wrote: > > > aaron.ballman wrote: > > >

  1   2   >