[PATCH] D144927: [GVNHoist] don't hoist callbr users into the callbr's block
Nick Desaulniers via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 1 16:49:26 PST 2023
nickdesaulniers added a comment.
In D144927#4163308 <https://reviews.llvm.org/D144927#4163308>, @nickdesaulniers wrote:
> Shouldn't `callbr` have it's own dedicated `MemoryDef` that's distinct from `liveOnEntry`?
Hmm..maybe not? We don't for vanilla inline asm (non-asm-goto) that modifies memory: https://godbolt.org/z/4144Wd7e1. Shouldn't inline asm qualify as a `MemoryAccess` when `"m"` constraints are used?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D144927/new/
https://reviews.llvm.org/D144927
More information about the llvm-commits
mailing list