<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>Issue</th>
<td>
<a href=https://github.com/llvm/llvm-project/issues/145286>145286</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>
[GitHub] Add description to llvm::instcombine label to cover instsimplify / vectorcombine as well
</td>
</tr>
<tr>
<th>Labels</th>
<td>
question,
llvm:instcombine,
github:workflow
</td>
</tr>
<tr>
<th>Assignees</th>
<td>
</td>
</tr>
<tr>
<th>Reporter</th>
<td>
RKSimon
</td>
</tr>
</table>
<pre>
CC @nikic @dtcxzyw
As explained here: https://llvm.org/docs/InstCombineContributorGuide.html#pick-the-correct-optimization-pass
Would it be OK to use the github label llvm::InstCombine to refer to the sibling passes as well:
```
InstructionSimplify: For folds that do not create new instructions (either fold to existing value or constant).
InstCombine: For folds that create or modify instructions.
AggressiveInstCombine: For folds that are expensive, or violate InstCombine requirements.
VectorCombine: For folds of vector operations that require target-dependent cost-modelling.
```
or would you prefer they all have separate labels?
The doc also refers to:
```
ConstantFolding: For folding instructions with constant arguments to a constant. (Mainly relevant for intrinsics.)
ValueTracking: For analyzing instructions, e.g. for known bits, non-zero, etc. Tests should usually use -passes=instsimplify.
```
But these probably should use the llvm::analysis label?
</pre>
<img width="1" height="1" alt="" src="http://email.email.llvm.org/o/eJx8VEGP4zYP_TXKhYjhyEl2fPDBm0H2-7AoCnQX7VmWaFsdWfRKdDKZX1_I9uxkFtMCAWJI5HvUeyRVjLbziJU4fBaHx42auKdQ_fH1mx3Ibxoyt-p0ArHPvX2yOn0Y1s8vtyuIvBZ5XUfA59Ep69FAjwFFUUPPPEZR1EKehTw7dxkyCp2QZ0M6Cnn-v498oqGxHk_kOdhmYgpfJmsw63lwQhaj1U9b7nGrKQTUvKWR7WBfFFvy21HFuPD_RZMzYBkahN-_AhNMEYF7hM5yPzXgVIMOUg2poKK-407RAVsM6SOlRNs46ztI8BhBRbiicylv5hLHfP3lNQBAggqTThV9s8PobHtLrz9TgJacicC9YjAEnhh0QMUIHq9g3_IiCPmAlntcclIl-GwjpzIuyk0IFECTj6w8C1lmifvuDR8QrkwUYCBj29s7vjm_7rqAMdoL_jeSCpjcRZ9ChTwlzIsll-DvdQz4Y7IBB_S8EADAn6iZwofY1MJlvgUaMahFh5lwBQJWoUPeGhzRG_QMmiJvBzLokkHZL2aIvKYA17kVbjTBuLra4w2Uc9CrC0LEUYVU-dwRURTnJfN7j2BIg3JxbYcITP9q-mn14kzOWN_dvyx59s7bq-X-p3mgQjfNEiWT1c_zLLXAb8p6d4OADi8ptqUANk2Gj1bHTMjyVdbUE9-D0k935Mord3v5lT4ZhlmXzWBPnq4eGsvzsSe_fcFAcwjrDL5j5AixnyWc4qScu82TtF1mQRSPCTqubf6BAZ8nToJHhDFQoxp3e4NbBvJtBud6o42LFaI4b0xVmLIo1Qar3adDfjwe83K_6StjcFfu2iY_7tqyKT_ti7yUB6ObMtelbPKNrWQuD_lRFvlD_rDbZ01uirJ9MLLV5rjbodjnOCjrstc1tLExTljt9gf5cNwszTCvPyl_TBiTdEJKIU9CyrXk9HS9dvLr1bJeRFFfKTy1jq7p5vC4CVVK2jZTF8U-dzZyfKNmy27etV8s_29qxOERamPAYNTBjok69cabUHfE6yJjAk0XDHBvBwh5XkfqNXrdXZspuOr9Ol4KzzQN625e_7ZjoL9Rs5DnWaG0qFeRLpX8JwAA__9g0RY0">