[Openmp-commits] [PATCH] D138704: RFC: [openmp] Provide an assembly implementation of __kmp_invoke_microtask on ARM

[David Spickett via Phabricator via Openmp-commits]

DavidSpickett added inline comments.


================
Comment at: openmp/runtime/src/z_Linux_asm.S:1447
+	// We strictly need 4*(argc-2) bytes, but allocate 4*argc for
+	// simplicity (to avoid needing to handle the argc<2 cases).
+	// We align the number of bytes allocated to 8 bytes, to keep the
----------------
What is the `-2` in `argc-2`? (or rather what would it be)


================
Comment at: openmp/runtime/src/z_Linux_asm.S:1452
+	// extra 8 bytes for gtid and tid.
+	mov	r5, #1
+	add	r5, r5, r3, lsr #1
----------------
This 1 here, is it part of the rounding or is it the extra 8 bytes for gtid and tid?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D138704/new/

https://reviews.llvm.org/D138704