[PATCH] D151626: [MemCpyOpt] clarify the comment in byval alignment test(NFC)
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun May 28 03:10:04 PDT 2023
nikic added a comment.
Based on the commit that introduced the test (https://github.com/llvm/llvm-project/commit/19e30d5a7d3369feae36cbe7f4f4c1f358881c41), this is indeed about the length arguments, but for a different transform. It's not intended to test the byval transform at all. I think it would be best to drop the `byval` attribute from this test.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D151626/new/
https://reviews.llvm.org/D151626
More information about the llvm-commits
mailing list