I have updated the patch with the suggested regex.
I have tested it here https://regex101.com/r/CZN3Mg/1
The e-mail "us\er2"@domain.com looks strange.

I have updated the patch to extract all e-mail addresses in the admin info.
Note the dot at the end of the e-mail address is alloved in case of the admin info like Name1 <user1@domain.com>, Name2 user2@domain.com.
email.utils.parseaddr keeps the dot at the end too. It seems email.utils.parseaddr uses a similar regex.

Updated the error message, removed the required role name (LLVM group name).

It seems already implemented here https://reviews.llvm.org/D132837

Ping

Ping

I have simplified the test and made it generic.

Why do you think the results of SROA should be the same with an without lifetime intrinsics?

Ping

I have updated the code and removed the loop from the test.

In D124967#3496117, @nikic wrote:

If I take your example with and without lifetime intrinsics, then with current SROA, the variant without intrinsics produces a worse result: https://llvm.godbolt.org/z/oPMKWzaMq
Results of SROA should be the same with an without lifetime intrinsics, but I'm not sure your patch achieves that -- it looks like it produces a third variant that is better than both?

Here is the result after this patch. Compare the while_body section.

define i16 @foo(i32* nocapture readonly %loop) #1 {
entry:
  br label %while_cond

I have updated the test. Now it is generic, not NVPTX specific.

You are right. I’ll look into this more.

I believe the problem is solved here https://reviews.llvm.org/D123702
I will check everything and resubmit this patch.