<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/REC-html40/loose.dtd">
<html lang="en" style="--code-editor-font: var(--default-mono-font, "GitLab Mono"), JetBrains Mono, Menlo, DejaVu Sans Mono, Liberation Mono, Consolas, Ubuntu Mono, Courier New, andale mono, lucida console, monospace;">
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
<title>
GitLab
</title>

<style data-premailer="ignore" type="text/css">
a { color: #1068bf; }
</style>

<style>img {
max-width: 100%; height: auto;
}
body {
font-size: .875rem;
}
body {
-webkit-text-shadow: hsla(0,0%,100%,.01) 0 0 1px;
}
body {
font-family: "GitLab Sans",-apple-system,BlinkMacSystemFont,"Segoe UI",Roboto,"Noto Sans",Ubuntu,Cantarell,"Helvetica Neue",sans-serif,"Apple Color Emoji","Segoe UI Emoji","Segoe UI Symbol","Noto Color Emoji"; font-size: inherit;
}
</style>
</head>
<body style="font-size: inherit; -webkit-text-shadow: hsla(0,0%,100%,.01) 0 0 1px; font-family: "GitLab Sans",-apple-system,BlinkMacSystemFont,"Segoe UI",Roboto,"Noto Sans",Ubuntu,Cantarell,"Helvetica Neue",sans-serif,"Apple Color Emoji","Segoe UI Emoji","Segoe UI Symbol","Noto Color Emoji";">
<div class="content">

<p style="color: #777777;">
<a href="https://gitlab.rtems.org/sebhub">Sebastian Huber</a>
<a href="https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1539#note_161209">commented</a>:
</p>
<div class="md" style="position: relative; z-index: 1; color: #3a383f; word-wrap: break-word;">
<p dir="auto" style="color: #3a383f; margin: 0px 0px 1rem;" align="initial">Thanks for the work on the LLVM support. My main concern is the rename of <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0px; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">RTEMS_GCOV_COVERAGE</code>, see the inline comments. Please keep the option name, the public function and the GCC log markers unchanged. The clang support can sit next to the GCC support under the same option. Commit <a href="https://gitlab.rtems.org/rtems/rtos/rtems/-/commit/d22ec37a1c91048c49cfb23a9b9f5b986469556d" title="spec,testsuites: rename RTEMS_GCOV_COVERAGE -> RTEMS_COVERAGE" class="gfm gfm-commit has-tooltip" data-original="d22ec37a1c9" data-link="false" data-link-reference="false" data-commit="d22ec37a1c91048c49cfb23a9b9f5b986469556d" data-project="26" data-project-path="rtems/rtos/rtems" data-container="body" data-placement="top" data-reference-type="commit" style="font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; font-size: 95%;">d22ec37a</a> can then go away, and the remaining coverage commits become smaller.</p>
<p dir="auto" style="color: #3a383f; margin: 0px 0px 1rem;" align="initial">Several commits fix or undo an earlier commit of this MR. Examples are <a href="https://gitlab.rtems.org/rtems/rtos/rtems/-/commit/f4b86d87381dd7acfdf061489971d441d721c0ee" title="cpukit: disabled gcov - clang does not have it" class="gfm gfm-commit has-tooltip" data-original="f4b86d87381" data-link="false" data-link-reference="false" data-commit="f4b86d87381dd7acfdf061489971d441d721c0ee" data-project="26" data-project-path="rtems/rtos/rtems" data-container="body" data-placement="top" data-reference-type="commit" style="font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; font-size: 95%; margin-top: 0px;">f4b86d87</a> and <a href="https://gitlab.rtems.org/rtems/rtos/rtems/-/commit/20bc773a2d13b5c460be45ea9136ad9ee2fdf00f" title="cpukit: Added LLVM coverage support, re-enable GCOV" class="gfm gfm-commit has-tooltip" data-original="20bc773a2d1" data-link="false" data-link-reference="false" data-commit="20bc773a2d13b5c460be45ea9136ad9ee2fdf00f" data-project="26" data-project-path="rtems/rtos/rtems" data-container="body" data-placement="top" data-reference-type="commit" style="font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; font-size: 95%;">20bc773a</a>, <a href="https://gitlab.rtems.org/rtems/rtos/rtems/-/commit/0ad50f5a8df5cdd5cdd6c21d2e08596ad95d6504" title="spec/build/bsps: Added sparc_q_compat as default for SPARC clang builds" class="gfm gfm-commit has-tooltip" data-original="0ad50f5a8df" data-link="false" data-link-reference="false" data-commit="0ad50f5a8df5cdd5cdd6c21d2e08596ad95d6504" data-project="26" data-project-path="rtems/rtos/rtems" data-container="body" data-placement="top" data-reference-type="commit" style="font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; font-size: 95%;">0ad50f5a</a> and <a href="https://gitlab.rtems.org/rtems/rtos/rtems/-/commit/be8daa6554d53dc3a5a2679593c1ae09c681729a" title="spec: Removed old sparc_q_compat lib" class="gfm gfm-commit has-tooltip" data-original="be8daa6554d" data-link="false" data-link-reference="false" data-commit="be8daa6554d53dc3a5a2679593c1ae09c681729a" data-project="26" data-project-path="rtems/rtos/rtems" data-container="body" data-placement="top" data-reference-type="commit" style="font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; font-size: 95%;">be8daa65</a>, <a href="https://gitlab.rtems.org/rtems/rtos/rtems/-/commit/013f82eaa78892dadeed9b2036ee452be0a06081" title="spec: Added preliminary specs for clang to skip couple of broken tests" class="gfm gfm-commit has-tooltip" data-original="013f82eaa78" data-link="false" data-link-reference="false" data-commit="013f82eaa78892dadeed9b2036ee452be0a06081" data-project="26" data-project-path="rtems/rtos/rtems" data-container="body" data-placement="top" data-reference-type="commit" style="font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; font-size: 95%;">013f82ea</a> and <a href="https://gitlab.rtems.org/rtems/rtos/rtems/-/commit/eef4f321e6dc5e298b3764835bb4934e198082f8" title="spec: Deleted unnecassary specs" class="gfm gfm-commit has-tooltip" data-original="eef4f321e6d" data-link="false" data-link-reference="false" data-commit="eef4f321e6dc5e298b3764835bb4934e198082f8" data-project="26" data-project-path="rtems/rtos/rtems" data-container="body" data-placement="top" data-reference-type="commit" style="font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; font-size: 95%;">eef4f321</a>, <a href="https://gitlab.rtems.org/rtems/rtos/rtems/-/commit/9b0da33efc80344fcf53a13151edfa8b3fbdb3e6" title="spec/build/bsps: Split linkcmds.yml into gcc and clang version" class="gfm gfm-commit has-tooltip" data-original="9b0da33efc8" data-link="false" data-link-reference="false" data-commit="9b0da33efc80344fcf53a13151edfa8b3fbdb3e6" data-project="26" data-project-path="rtems/rtos/rtems" data-container="body" data-placement="top" data-reference-type="commit" style="font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; font-size: 95%;">9b0da33e</a> and <a href="https://gitlab.rtems.org/rtems/rtos/rtems/-/commit/a46641bd0d92f429b7434631d3b79b8850c109b6" title="linkcmds-clang.yml: Remove python logic form linkcmds-clang" class="gfm gfm-commit has-tooltip" data-original="a46641bd0d9" data-link="false" data-link-reference="false" data-commit="a46641bd0d92f429b7434631d3b79b8850c109b6" data-project="26" data-project-path="rtems/rtos/rtems" data-container="body" data-placement="top" data-reference-type="commit" style="font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; font-size: 95%;">a46641bd</a>, and <a href="https://gitlab.rtems.org/rtems/rtos/rtems/-/commit/5505bbdef68b111772a71e69bbb67e09dfc20c49" title="cpukit: Fixed typo RTEMSImplCcov -> RTEMSImplCov" class="gfm gfm-commit has-tooltip" data-original="5505bbdef68" data-link="false" data-link-reference="false" data-commit="5505bbdef68b111772a71e69bbb67e09dfc20c49" data-project="26" data-project-path="rtems/rtos/rtems" data-container="body" data-placement="top" data-reference-type="commit" style="font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; font-size: 95%;">5505bbde</a>. Please squash them, so that each commit builds and stands on its own.</p>
<p dir="auto" style="color: #3a383f; margin: 0px 0px 1rem;" align="initial">Some hunks reformat lines which they do not change, for example in <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0px; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">rtl-mdreloc-sparc.c</code>, <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">atomic.h</code> and <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">fstests/fserror/test.c</code>. Please keep the style of each file.</p>
<p dir="auto" style="color: #3a383f; margin: 0px 0px 1rem;" align="initial">The fixes in <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0px; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">psx13</code>, <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">termios02</code>, <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">default_lseek_file.c</code> and <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">tc-score-isr.c</code> do not depend on clang. A separate MR for them can go in quickly. The fix in <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">tc-score-isr.c</code> is correct. GCC supports a local register variable only as an operand of an extended asm statement.</p>
<p dir="auto" style="color: #3a383f; margin: 0px;" align="initial">The failed pipeline is no fault of this MR. All four jobs stopped on the runner when they fetched the source. Two of them report <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0px; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">No space left on device</code>.</p>
</div>


</div>
<div class="footer" style="margin-top: 10px;">
<p style="font-size: small; color: #626168;">
—
<br>
<a href="https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1539#note_161209">View it on GitLab</a>.
<br>
You're receiving this email because of your account on <a target="_blank" rel="noopener noreferrer" href="https://gitlab.rtems.org">gitlab.rtems.org</a>. <a href="https://gitlab.rtems.org/-/namespace/49/sent_notifications/6-eepoaz22v5bo7ulw9rvdy6tmo-1d/unsubscribe" target="_blank" rel="noopener noreferrer">Unsubscribe</a> from this thread · <a href="https://gitlab.rtems.org/-/profile/notifications" target="_blank" rel="noopener noreferrer" class="mng-notif-link">Manage all notifications</a> · <a href="https://gitlab.rtems.org/help" target="_blank" rel="noopener noreferrer" class="help-link">Help</a>
<span style="color: transparent; font-size: 0; display: none; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0;">
Notification message regarding https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1539#note_161209 at 1792129047
</span>
<script type="application/ld+json">{"@context":"http://schema.org","@type":"EmailMessage","action":{"@type":"ViewAction","name":"View Merge request","url":"https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1539#note_161209"}}</script>


</p>
</div>
</body>
</html>