Skip to content

Commit

Permalink
address review feedback on doc changes
Browse files Browse the repository at this point in the history
Signed-off-by: Jeremiah Corrado <[email protected]>
  • Loading branch information
jeremiah-corrado committed Sep 18, 2023
1 parent c297b2c commit 79982a0
Show file tree
Hide file tree
Showing 6 changed files with 12 additions and 8 deletions.
3 changes: 2 additions & 1 deletion modules/dists/BlockDist.chpl
Original file line number Diff line number Diff line change
Expand Up @@ -328,7 +328,8 @@ The helper methods on ``blockDist`` have the following signatures:
.. function:: proc blockDist.createDomain(dom: domain(?))
Create a block-distributed domain over an existing ``blockDist``.
Create a block-distributed domain over an existing ``blockDist`` by copying
the index space from the passed domain.
.. function:: proc blockDist.createDomain(rng: range(?)...)
Expand Down
3 changes: 2 additions & 1 deletion modules/dists/CyclicDist.chpl
Original file line number Diff line number Diff line change
Expand Up @@ -230,7 +230,8 @@ The helper methods on ``Cyclic`` have the following signatures:
.. function:: proc cyclicDist.createDomain(dom: domain(?))
Create a cyclic-distributed domain over an existing ``cyclicDist``.
Create a cyclic-distributed domain over an existing ``cyclicDist`` by copying
the index space from the passed domain.
.. function:: proc cyclicDist.createDomain(rng: range(?)...)
Expand Down
3 changes: 2 additions & 1 deletion modules/dists/StencilDist.chpl
Original file line number Diff line number Diff line change
Expand Up @@ -266,7 +266,8 @@ config param disableStencilLazyRAD = defaultDisableLazyRADOpt;
.. function:: proc stencilDist.createDomain(dom: domain(?))
Create a stencil-distributed domain over an existing ``blockDist``.
Create a stencil-distributed domain over an existing ``blockDist`` by copying
the index space from the passed domain.
.. function:: proc stencilDist.createDomain(rng: range(?)...)
Expand Down
4 changes: 2 additions & 2 deletions modules/packages/LinearAlgebra.chpl
Original file line number Diff line number Diff line change
Expand Up @@ -3480,11 +3480,11 @@ module Sparse {


@chpldoc.nodoc
/* Returns ``true`` if the array is mapped to the ``CS`` layout. */
/* Returns ``true`` if the array is distributed with the CS layout. */
proc isCSArr(A: []) param { return isCSType(A.domain.distribution.type); }

@chpldoc.nodoc
/* Returns ``true`` if the domain is mappd to the ``CS`` layout. */
/* Returns ``true`` if the domain is distributed with the CS layout. */
proc isCSDom(D: domain) param { return isCSType(D.distribution.type); }


Expand Down
2 changes: 1 addition & 1 deletion test/release/examples/primers/distributions.chpl
Original file line number Diff line number Diff line change
Expand Up @@ -189,7 +189,7 @@ following expressions for convenience:
const CyclicSpace = cyclicDist.createDomain(Space);
const CA: [CyclicSpace] int;
or:c
or:
.. code-block:: chapel
Expand Down
5 changes: 3 additions & 2 deletions test/release/examples/primers/forallLoops.chpl
Original file line number Diff line number Diff line change
Expand Up @@ -153,12 +153,13 @@ determine iteration locality.
Domains declared without a distribution (see :ref:`primers-distributions`),
including default rectangular and default associative domains,
as well as arrays over such domains, provide both serial and parallel
iterators. So do domains distributed over standard distributions,
iterators. So do domains distributed over standard multi-locale distributions,
such as blockDist and cyclicDist, and arrays over such domains. The
parallel iterators provided by standard distributions place each loop
parallel iterators provided by standard multi-locale distributions place each loop
iteration on the locale where the corresponding index or array element
is placed.
Task Intents and Shadow Variables
---------------------------------
Expand Down

0 comments on commit 79982a0

Please sign in to comment.