[PATCH] D137791: [SDAG] bail out of mergeTruncStores() if there's an unknown store in the chain

Jonas Paulsson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Nov 12 06:01:53 PST 2022


jonpa added a comment.

As I mentioned on the other thread, it would probably be better to derive a test case from main if possible, where the bug is exposed (64-bit overlapping store after store of 0).

Maybe stop after ISel?  Should all the unused metadata and datalaoyut string be included? (I usually strip that away for readability personally...)


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

https://reviews.llvm.org/D137791



More information about the llvm-commits mailing list