<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">