vhscampos wrote:
I've addressed your comments, but I don't have permissions to edit this PR
directly. Thus I've created a new pull request:
https://github.com/llvm/llvm-project/pull/78305
Can you please have a look? Thanks
https://github.com/llvm/llvm-project/pull/2
__
@@ -32,7 +32,7 @@ extern "C" {
#define __isb(i) __builtin_arm_isb(i)
#endif
-/* 8.4 Hints */
+/* 7.4 Hints */
voltur01 wrote:
HTML version has the extra `.`
https://arm-software.github.io/acle/main/acle.html#hints but the PDF one does
not
https://github.co
@@ -20,8 +20,8 @@
extern "C" {
DavidSpickett wrote:
You may want to note that section numbers were correct at time of writing, if
they've changed then look for the section title.
https://github.com/llvm/llvm-project/pull/2
_
@@ -32,7 +32,7 @@ extern "C" {
#define __isb(i) __builtin_arm_isb(i)
#endif
-/* 8.4 Hints */
+/* 7.4 Hints */
DavidSpickett wrote:
If we're matching the section titles then they should have the extra `.` on the
end of the number, so here `7.4. Hints`. Just t
https://github.com/DavidSpickett commented:
If these section refs weren't already here I'd suggest not adding numbers at
all given that they will change later, and Firefox and Chrome seem hopeless at
searching for them in the HTML document.
But we have them already so I won't suggest that :) T
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/2
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -20,8 +20,8 @@
extern "C" {
vhscampos wrote:
Thanks @Blue-Dot for the patch. I agree with @kbeyls that it would be a good
idea.
https://github.com/llvm/llvm-project/pull/2
___
cfe-commits mailing list
cfe-comm
@@ -20,8 +20,8 @@
extern "C" {
kbeyls wrote:
Thanks, all of these updates look correct to me.
Since these all refer to sections in a document, I think it may be useful too
to add a hyperlink at the top of the document to where the ACLE lives.
I think the correc
llvmbot wrote:
@llvm/pr-subscribers-backend-x86
Changes
Non functional change to update section comments in arm_acle.h, in order to
align with updated documentation:
[https://arm-software.github.io/acle/main/acle.html](https://arm-software.github.io/acle/main/acle.html)
---
Full diff: h
https://github.com/Blue-Dot created
https://github.com/llvm/llvm-project/pull/2
Non functional change to update section comments in arm_acle.h, in order to
align with updated documentation:
[https://arm-software.github.io/acle/main/acle.html](https://arm-software.github.io/acle/main/acle.
10 matches
Mail list logo
