|
7 | 7 | //===----------------------------------------------------------------------===// |
8 | 8 | // Estimate if conditional branches for which SIAnnotateControlFlow introduced |
9 | 9 | // amdgcn_if or amdgcn_else intrinsics are likely to have different outcomes for |
10 | | -// the threads of each wavefront. If that is the case, BranchWeight metadata is |
| 10 | +// the lanes of each wavefront. If that is the case, BranchWeight metadata is |
11 | 11 | // added to signal that "then" and "else" blocks are both likely to be executed. |
12 | 12 | // This may introduce branch weights that would be self-contradictory in a |
13 | 13 | // non-SIMT setting. |
14 | 14 | // |
15 | 15 | // A consequence of this is that SIPreEmitPeephole is more likely to eliminate |
16 | 16 | // s_cbranch_execz instructions that were introduced to skip these blocks when |
17 | | -// no thread in the wavefront is active for them. |
| 17 | +// no lane in the wavefront is active for them. |
18 | 18 | // |
19 | 19 | // Should only run after SIAnnotateControlFlow. |
20 | 20 | //===----------------------------------------------------------------------===// |
@@ -218,7 +218,7 @@ bool AMDGPUAnnotateVaryingBranchWeightsImpl::run(Function &F) { |
218 | 218 |
|
219 | 219 | // reqd_work_group_size determines the size of the work group in every |
220 | 220 | // dimension. If it is present, identify the dimensions where the workitem id |
221 | | - // differs between the threads of the same wavefront. Otherwise assume that |
| 221 | + // differs between the lanes of the same wavefront. Otherwise assume that |
222 | 222 | // only dimension 0, i.e., x, varies. |
223 | 223 | // |
224 | 224 | // TODO can/should we assume that workitems are grouped into waves like that? |
|
0 commit comments