[PATCH] D114832: [SROA] Improve SROA to prevent generating redundant coalescing operations.

[Mingming Liu via Phabricator via llvm-commits]

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
----------------
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?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114832