@danilaml OrderedRoots has Instructions from the same BasicBlock, and they get there one by one during for (auto &I : *B) iteration. So, there is no risk of messing up there unless someone tries to add much more functionality :)

Added tests with different store pointers

Add tests with different store pointers

Ping about the RISC-V conversation and more test requirements

Please make sure to pre-commit test coverage (https://llvm.org/docs/TestingGuide.html#precommit-workflow-for-tests). This is also missing some tests, in particular multi-use and negative tests.

• Added test with @use applied to nested add
• Minor fixes

Added reference to the github issue

Added a test for neoverse-v1

Remove RUN with -mcpu=neoverse-v1 due to too many changes

Rebase after adding extra tests

• Add RUN line to sve-ld1r.ll test with -mcpu=neoverse-v1
• Update feature description
• Add missing Predicate

Renaming

Update CHECK lines

• Rename new feature avoid-ldr1 to no-sve-fp-ld1r
• Now change is applied only to FP LD1R instructions
• Some other renaming inside code

Update CHECK lines

Update test functions names

• Vscale -> VScale
• Move foldVScaleSplatAddSub call to visitAdd and visitSub
• Refactor pattern detection algorithm

Rebase

Remove comment. Update tests to match patterns better

Replace AArch64ptrue 31 pattern with SVEAllActive

@NickGuy, it looks like we had O(2**N) complexity there where N is the depth of identifyNode call stack :)

Added link to GitHub issue

Move code to a separate function and completely refactor it

Update commit message and some tests comments

Move comments from definition to declaration

Address comments

Rebase and update commit message

I adjusted the commit message rather than adding a test. It doesn't matter if vectors are scalable or fixed-width cause code that processes PHINodes does not behave differently. So I think adding a scalable test doesn't add more coverage.

Update test and commit message

Update test and commit message

@glandium, thanks. I've reproduced and prepared the solution - https://reviews.llvm.org/D154598

Add argument name

Ok, then!

Rebase after pre-commit

Forgot to run tests update script

• IsOperationsupported -> IsOperationSupported
• Remove some artefacts