[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