<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>Issue</th>
<td>
<a href=https://github.com/llvm/llvm-project/issues/79249>79249</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>
Inquiry Regarding Exclusion of Super Alignment for DynLDS in D115488
</td>
</tr>
<tr>
<th>Labels</th>
<td>
new issue
</td>
</tr>
<tr>
<th>Assignees</th>
<td>
</td>
</tr>
<tr>
<th>Reporter</th>
<td>
Origami404
</td>
</tr>
</table>
<pre>
Dear @JonChesterfield,
I hope this issue finds you well. I am an enthusiast developer interested in LLVM AMDGPU backend, and I noticed that in your patch [D115488](https://reviews.llvm.org/D115488) submitted to LLVM, you excluded handling super alignment for LDS. I am curious about the rationale behind this change and would appreciate if you could provide some insights into your decision.
More precisely, I want to understand why you included this code: (still existing in LLVM at [AMDGPULowerModuleLDSPass.cpp#L1339](https://github.com/llvm/llvm-project/blob/baba7e4175b6ca21e83b1cf8229f29dbba02e979/llvm/lib/Target/AMDGPU/AMDGPULowerModuleLDSPass.cpp#L1339) currently)
```cpp
if (!GV.hasInitializer()) {
// cuda/hip extern __shared__ variable, leave alignment alone
continue;
}
```
(Since LLVM Phabricator has been closed, I can only raise this question in Github issue. )
</pre>
<img width="1px" height="1px" alt="" src="http://email.email.llvm.org/o/eJyUVF-P4jYQ_zTmZVSUOIGQBx52L2VFxaqrbnuvq0k8SaY1ds5_YOmnr5xw172qUlWECLKcmd-_GfSeB0O0F5tHsWlWGMNo3f5nxwOeuczKVWvVbd8QOhBl9pM1n0bygVzPpJWQn0TWiOxh-T3CaCeCMLIH9j4S9GyUh5uNcCWt13AEPAMaIBPG6Bl9AEUX0nYiB2wCuVRcARs4nT4_w8Nz8_TyG7TY_UEmtQM0Co5gbOCOFIQRQ7p8s9HBhKEbQWwemzzflLud2DRC7sYQJi-KByEPQh4cXZiufq315by2bhDy8PW2rMHH9swhAQh2BpA6JvT03umoSMGIRmk2A_iYIKPmwZzJBOitg1PzeqfYRcc2esDWxgBhJHAY2BrUBC2NbNSiUjeiGWgmdbVRK8BpctQxBgLu59bdfD45e2FF4O2ZgI3nYQw-KWYX7oo69mzN-qMfz9YRzPU86VvicoQrmpDYRaPI-TB3Hm9zJzZ3kgs0q0gUDyDkzgfWGuidfUjUv5qDIYm9WHSyV3LPVkVNp-b1Bb1fd9MkZHHKi6L-NycGDmNs1509C3lIdtwfP0zO_k5dEPLQatumB7ZYUZlXm3bbocxpV7R51--krHtZq7bFTFJd1R_qcHrvV3QDpToLxG9__gOrrJN9jkxImtV3QbfZ8k035xOYP9wngYTMnz6vR_RHw4FR85_k5uO5mqgeP74CsAgAXVQo5GHkCeg9kDPw9uZHdKTe3uCCjrHVlFzThBf6kDXU1tC9ZGdNYBNJFN81EVXzD-AfgyHk7pVNR4uPLyO2jjsM1sGIHloiA522ntSSmQ4NWKNv4JD9fb6_RPIp0SkNT7OVy8ivQch6pfaFqosaV7TPq2ybbausqFfjvs6qbVvXxbbfyl3dZ7Tb5lW1LbO2yKoc1Yr3MpNllssyK4pMbtfljkrCDbZ5WfWZ2ogyozOy_jbBq7ntvqplWa80tqT9vMykNHRdMAkp025z-zlfbRy8KDPNPvy9B1aBg6b90XyJ7G7wCw3oVAr7j2ny02SB7eF1HvqH74a-uZlT85pkuC-SVXR6_7-zPgP1Qh5mIn8FAAD___oXyus">