[llvm] [llvm][loop-rotate] Allow forcing loop-rotation (PR #82828)

Paul Kirth via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 26 10:00:02 PST 2024


ilovepi wrote:

> Could you add a PhaseOrdering test that shows the impact of the change? Possibly just with the reproducer from #50308

@fhahn I've added that test case. I used `udate_test_checks.py`, which seems to be the norm in this directory, but TBH it seems like some of the cases are a bit brittle, as its checking for many attributes that aren't relevant. It could probably just be a check for `memset` or not. Is there a preference in these tests for a hand maintained test vs auto-generated ones?

https://github.com/llvm/llvm-project/pull/82828


More information about the llvm-commits mailing list