[PATCH] D71547: Resubmit "[Alignment][NFC] Deprecate CreateMemCpy/CreateMemMove"

Tue Feb 25 06:39:33 PST 2020

gchatelet added a comment.

In D71547#1889865 <https://reviews.llvm.org/D71547#1889865>, @aganea wrote:

> This occurs in trunk and the 10.0 branch as well. @gchatelet Would you have a chance to take a look?

I could reproduce the failing assert with a clang built at rGbdbb894a52997c1679d7f716397435c95e637fb3 <https://reviews.llvm.org/rGbdbb894a52997c1679d7f716397435c95e637fb3> (Apr 20 2017) so this is not a regression but a latent bug that has been uncovered.
Please have a look at the bug for a more detailed answer.

I'm not sure how far the reduced example is from the real code but I suspect that using an `auto&` instead of `auto` for the `foreach` variable will fix the issue.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71547/new/

https://reviews.llvm.org/D71547