<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="">
<div style="color: black; font-family: Calibri, Arial, Helvetica, sans-serif, serif, EmojiFont; font-size: 12pt; margin: 0px; background-color: white;">
<span style="margin: 0px; font-size: 14.67px; font-family: "Times New Roman", serif, EmojiFont; color: rgb(50, 49, 48); background-color: white">Hi James,</span><br>
<span style="margin: 0px; font-size: 14.67px; font-family: "Times New Roman", serif, EmojiFont; color: rgb(50, 49, 48); background-color: white">Could you have a look at this patch and provide a senior review please. The code isn't mine ðŸ™‚ (Mikhail is the original
 author for the downstream patches)</span><br>
<br>
<span style="margin: 0px; font-size: 14.67px; font-family: "Times New Roman", serif, EmojiFont; color: rgb(50, 49, 48); background-color: white">Requesting Senior review, This patch upstreams the mcpu option for Cortex-A78, Cortex-X1 is now public and outlined
 here:</span><br>
<br>
<a href="https://www.arm.com/products/silicon-ip-cpu/cortex-a/cortex-a78" id="LPNoLP108940">https://www.arm.com/products/silicon-ip-cpu/cortex-a/cortex-a78</a></div>
<div style="color: black; font-family: Calibri, Arial, Helvetica, sans-serif, serif, EmojiFont; font-size: 12pt; margin: 0px; background-color: white;">
<br>
</div>
<div style="color: black; font-family: Calibri, Arial, Helvetica, sans-serif, serif, EmojiFont; font-size: 12pt; margin: 0px; background-color: white;">
and </div>
<div style="color: black; font-family: Calibri, Arial, Helvetica, sans-serif, serif, EmojiFont; font-size: 12pt; margin: 0px; background-color: white;">
<br>
</div>
<div style="color: black; font-family: Calibri, Arial, Helvetica, sans-serif, serif, EmojiFont; font-size: 12pt; margin: 0px; background-color: white;">
<br>
<a href="https://www.arm.com/products/cortex-x" id="LPNoLP179828">https://www.arm.com/products/cortex-x</a><br>
<br>
<br>
<span style="margin: 0px; font-size: 14.67px; font-family: "Times New Roman", serif, EmojiFont; color: rgb(50, 49, 48); background-color: white">Questionnaire is below</span><br>
<br>
<span style="margin: 0px; font-size: 14.67px; font-family: "Times New Roman", serif, EmojiFont; color: rgb(50, 49, 48); background-color: white">The cpus themselves are public, but clang/llvm support is not yet upstream, this changes that. Below is a summary
 of changes (compared to downstream), and questions I can't answer.</span><br>
<br>
<span style="margin: 0px; font-size: 14.67px; font-family: "Times New Roman", serif, EmojiFont; color: rgb(50, 49, 48); background-color: white">I shall use the git-format patch -U9999 argument when I upstream, for this email I'm keeping it short with -U10</span><br>
<br>
<span style="margin: 0px; font-size: 14.67px; font-family: "Times New Roman", serif, EmojiFont; color: rgb(50, 49, 48); background-color: white">---Changes---</span><br>
<span style="margin: 0px; font-size: 14.67px; font-family: "Times New Roman", serif, EmojiFont; color: rgb(50, 49, 48); background-color: white"> - One of the unittests weirdly fails when applying this on vanilla llvm, I am debugging this now but it shouldn't
 have a bearing on senior review, will fix and self approve that before I upstream<br>
 - as we were discussing on dsg-gnu slack, we can't find a clickable link to the X1 TRM to get the MIDR numbers for Cortex-X1. We can see these in the search results of devloper.arm.com instead. I have added them to the code but it should be discussed here
 whether we can include this upstream. For my two cents, MIDR numbers are a safe thing to upstream without any issue as it's just a part number</span></div>
<div style="margin: 0px; background-color: white;"><br>
<span style="color: rgb(50, 49, 48); font-family: "Times New Roman", serif, EmojiFont; font-size: 14.67px; margin: 0px; background-color: white;">---Questions--- </span><br>
</div>
<div style="color: rgb(32, 31, 30); font-family: "Segoe UI", "Segoe UI Web (West European)", "Segoe UI", -apple-system, BlinkMacSystemFont, Roboto, "Helvetica Neue", sans-serif; font-size: 15px; margin: 0px; background-color: white;">
</div>
<div style="color: black; font-family: Calibri, Arial, Helvetica, sans-serif, serif, EmojiFont; font-size: 12pt; margin: 0px; background-color: white;">
<span style="margin: 0px; font-size: 14.67px; font-family: "Times New Roman", serif, EmojiFont; color: rgb(50, 49, 48); background-color: white">no outstanding concerns<br>
</span><br>
<span style="margin: 0px; font-size: 14.67px; font-family: "Times New Roman", serif, EmojiFont; color: rgb(50, 49, 48); background-color: white">---Questionnaire---</span><br>
<span style="margin: 0px; font-size: 14.67px; font-family: "Times New Roman", serif, EmojiFont; color: rgb(50, 49, 48); background-color: white"></span><span style="margin: 0px; font-family: "Times New Roman", serif, EmojiFont; background-color: white">Q0.
 Is it based on previously approved IP with minor changes? no</span><br>
<span style="margin: 0px; font-family: "Times New Roman", serif, EmojiFont; background-color: white">Q1. Are some changes in this piece of IP not written by you? yes, downstream contributions made by other Arm employees</span><br>
<span style="margin: 0px; font-family: "Times New Roman", serif, EmojiFont; background-color: white">Q2. Does this disclose anything that is not public knowledge? Or does this patch introduce new architectural or core support? yes. This adds new CPU support,
 support for which does not yet exist in GCC</span></div>
<div style="color: black; font-family: "Segoe UI", "Segoe UI Web (West European)", "Segoe UI", -apple-system, BlinkMacSystemFont, Roboto, "Helvetica Neue", sans-serif; font-size: 12pt; margin: 0px; background-color: white;">
<br>
</div>
<div style="color: black; font-family: "Segoe UI", "Segoe UI Web (West European)", "Segoe UI", -apple-system, BlinkMacSystemFont, Roboto, "Helvetica Neue", sans-serif; font-size: 12pt; margin: 0px; background-color: white;">
<span style="margin: 0px; font-family: "Times New Roman", serif, EmojiFont; background-color: white">Q3. Does this mention any company name other than Arm? no</span><br>
<span style="margin: 0px; font-family: "Times New Roman", serif, EmojiFont; background-color: white">Q4. Does this use any trademark under inappropriate circumstances? no</span><br>
<span style="margin: 0px; font-family: "Times New Roman", serif, EmojiFont; background-color: white">Q5. Does this mention or imply any sensitive benchmark score? no</span><br>
<span style="margin: 0px; font-family: "Times New Roman", serif, EmojiFont; background-color: white">Q6. Does this relate to any micro-architectural aspects of any Arm IP? no</span><br>
<span style="margin: 0px; font-family: "Times New Roman", serif, EmojiFont; background-color: white">Q7. Is it related to any not yet published standards content or specification? no</span><br>
<span style="margin: 0px; font-family: "Times New Roman", serif, EmojiFont; background-color: white">Q8. Does it reveal a patentable idea for which a patent has not yet been filed? no</span><br>
<span style="margin: 0px; font-family: "Times New Roman", serif, EmojiFont; background-color: white">Q9. Does it interpret assembler instructions? no</span><br>
<span style="margin: 0px; font-family: "Times New Roman", serif, EmojiFont; background-color: white">Q10. Is this a patch contributing beyond the scope that has been approved for the specific sub-project? no</span><br>
<span style="margin: 0px; font-family: "Times New Roman", serif, EmojiFont; background-color: white">Q11. Should Arm be the key player defining this change? yes</span><br>
<span style="margin: 0px; font-family: "Times New Roman", serif, EmojiFont; background-color: white">Q12. Does this contribution have an unfulfilled submission sequence dependency on projects with a different license? no</span><br>
<span style="margin: 0px; font-family: "Times New Roman", serif, EmojiFont; background-color: white">Q13. Does your contribution fall within the boundaries of business approval? yes</span><br>
<span style="margin: 0px; font-family: "Times New Roman", serif, EmojiFont; background-color: white">Q20. Do you have any doubt about letting this go outside Arm, or do you think others might? no</span><br>
<span style="margin: 0px; font-size: 14.67px; font-family: "Times New Roman", serif, EmojiFont; color: rgb(50, 49, 48); background-color: white">See commit message for attribution</span><br>
<br>
<span style="margin: 0px; font-size: 14.67px; font-family: "Times New Roman", serif, EmojiFont; color: rgb(50, 49, 48); background-color: white">---Commit Msg---</span></div>
<span>Subject: [PATCH 2/2] [ARM] Add Cortex-A78 and Cortex-X1 Support for Clang and<br>
</span>
<div> LLVM<br>
</div>
<div><br>
</div>
<div>This patch upstreams support for the Arm-v8 Cortex-A78 and Cortex-X1<br>
</div>
<div>processors for AArch64 and ARM.<br>
</div>
<div><br>
</div>
<div>In detail:<br>
</div>
<div>- Adding cortex-a78 and cortex-x1 as cpu options for aarch64 and arm targets in clang<br>
</div>
<div>- Adding Cortex-A78 and Cortex-X1 CPU names and ProcessorModels in llvm<br>
</div>
<div><br>
</div>
<div>details of the CPU can be found here:<br>
</div>
<div>https://www.arm.com/products/cortex-x<br>
</div>
<div><br>
</div>
<div>https://www.arm.com/products/silicon-ip-cpu/cortex-a/cortex-a78<br>
</div>
<div><br>
</div>
<div>The following people contributed to this patch:<br>
</div>
<div>- Luke Geeson<br>
</div>
<span>- Mikhail Maltsev</span><br>
</div>
</body>
</html>