[PATCH] D114832: [SROA] Improve SROA to prevent generating redundant coalescing operations.
Mingming Liu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 2 14:41:09 PST 2021
luna marked 2 inline comments as done.
luna added inline comments.
================
Comment at: llvm/test/Transforms/SROA/alloca-struct.ll:65
+; COM: Check that there is only PHI node to select the whole value.
+; CHECK-COUNT-1: phi
; CHECK: ret i64
----------------
aeubanks wrote:
> luna wrote:
> > nikic wrote:
> > > Please use `update_test_checks.py`.
> > Thanks for the suggestions!
> >
> > Using `update_test_checks.py` might make the test case a little fragile. This is a preview of the result (https://gist.github.com/minglotus-6/a05ac0300d8ea8204ec9eed409951b25)
> >
> > To improve the current test style, maybe I can enrich it a little bit but still scrub updater output that look strict?
> it's very hard to tell what changed without seeing every instruction in the function, so I'd also recommend `update_test_checks.py`. yes it's fragile, but that's kind of the point, to be able to see changes in IR
Done.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D114832/new/
https://reviews.llvm.org/D114832
More information about the llvm-commits
mailing list