[PATCH] D99784: [LICM] Hoist loads with invariant.group metadata
Arthur Eubanks via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 31 09:18:28 PDT 2021
aeubanks added a comment.
No, the current patch is still wrong. We can end up hoisting a load before the corresponding store. We currently don't check for that condition when the invariant.group metadata is present.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D99784/new/
https://reviews.llvm.org/D99784
More information about the llvm-commits
mailing list