In D68997#2377641, @stella.stamenova wrote:

This change broke the windows lldb bot:

http://lab.llvm.org:8011/#/builders/83/builds/570

Can you please fix this or revert?

Fixed the trailing whitespace.

Rebased on ToT.

Thanks a lot.

Oh I think you'd need to edit the revision on Phabricator (top right on this page, edit revision). But if you are fine with it I can commit the patch for you with the adjusted commit title.

In D88265#2302653, @fhahn wrote:

Could you adjust the commit message to be a bit more descriptive, e.g something like [Sema] Support Comma operator for fp16 vectors.

Update commit message.

I do not have commit rights, so it would be great if you can land it.

Address review comments.

Addressed review comments.

Fix a typo.

Addressed review comments.

Rebase.

Rebase on top-of-tree.

Could you land it for me Mehdi? I do not have permissions.

Rebase on top of tree.

Rebase on top of tree.

Apply clang-format.

Rename and clarify.

In D72100#1855483, @jkorous wrote:

We should either simplify the implementation to reflect that we don't support e. g. *:42 (seems preferable to me) or have the codepaths that are currently not accessible through -fverify tested by other means.

Address review comments.

Ping.

This is for example useful to add a catch-all clause like // expected-note-re@*:* 1+ {{candidate function {{.+}}}}

Ping.
Still looking for someone to take a look. Happy to answer any questions.

Ping.

Rebase.

Following some review with Ahmed (ab), we think it will not make sense for upstream.

Rebased on top of tree.