[clang] 0fc27ef - [clang-format] handle trailing comments in function definition detection

2021-08-12 Thread Krasimir Georgiev via cfe-commits
Author: Krasimir Georgiev Date: 2021-08-12T08:55:54+02:00 New Revision: 0fc27ef19670676689d3317948c81eb171bb25f8 URL: https://github.com/llvm/llvm-project/commit/0fc27ef19670676689d3317948c81eb171bb25f8 DIFF:

[PATCH] D106738: [RISCV] Use getNaturalPointeeTypeAlignment to get alignment for stores created for vector builtins.

2021-08-12 Thread Hsiangkai Wang via Phabricator via cfe-commits
HsiangKai accepted this revision. HsiangKai added a comment. This revision is now accepted and ready to land. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106738/new/ https://reviews.llvm.org/D106738

[PATCH] D107907: [clang-format] handle trailing comments in function definition detection

2021-08-12 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir updated this revision to Diff 365918. krasimir added a comment. - apply reviewer suggestion Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107907/new/ https://reviews.llvm.org/D107907 Files: clang/lib/Format/TokenAnnotator.cpp

[PATCH] D107946: [X86] Reverse *_set_ph and *_setr_ph 's set order.

2021-08-12 Thread Craig Topper via Phabricator via cfe-commits
craig.topper added inline comments. Comment at: clang/lib/Headers/avx512fp16intrin.h:92 #define _mm512_setr_ph(__h1, __h2, __h3, __h4, __h5, __h6, __h7, __h8, __h9, \ __h10, __h11, __h12, __h13, __h14, __h15, __h16, __h17, \ Can we

[PATCH] D107946: [X86] Reverse *_set_ph and *_setr_ph 's set order.

2021-08-12 Thread LuoYuanke via Phabricator via cfe-commits
LuoYuanke added a comment. Any test case update? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107946/new/ https://reviews.llvm.org/D107946 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D107946: [X86] Reverse *_set_ph and *_setr_ph 's set order.

2021-08-12 Thread Pengfei Wang via Phabricator via cfe-commits
pengfei accepted this revision. pengfei added a comment. This revision is now accepted and ready to land. LGTM. Thanks Freddy. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107946/new/ https://reviews.llvm.org/D107946

[PATCH] D107641: [clang-tidy] fix duplicate '{}' in cppcoreguidelines-pro-type-member-init

2021-08-12 Thread gehry via Phabricator via cfe-commits
Sockke updated this revision to Diff 365916. Sockke added a comment. update! @aaron.ballman @MTC CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107641/new/ https://reviews.llvm.org/D107641 Files: clang-tools-extra/clang-tidy/cppcoreguidelines/ProTypeMemberInitCheck.cpp

[PATCH] D107946: [X86] Reverse *_set_ph and *_setr_ph 's set order.

2021-08-12 Thread Freddy, Ye via Phabricator via cfe-commits
FreddyYe created this revision. FreddyYe requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D107946 Files: clang/lib/Headers/avx512fp16intrin.h

<    1   2   3