[llvm] [coro][pgo] Remove redundant coroutine test files (PR #89620)
    via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Apr 22 10:08:19 PDT 2024
    
    
  
zmodem wrote:
> This was intentional, see the comment that introduced them ([#71262 (review)](https://github.com/llvm/llvm-project/pull/71262#pullrequestreview-1723261241))
I saw that, but I don't agree with the outcome :-)
> An advantage is that the tests cover 2 concerns, really: coro lowering, and PGO instrumentation. They can potentially diverge if needed.
The original tests (e.g. llvm/test/Transforms/Coroutines/coro-split-musttail.ll) already cover both concerns nicely.
If there's a need to diverge, new tests could be created when that need arises.
> Other than needing to duplicate a fix in both places, is there any other shortcoming of the double presence?
Needing to update two almost identical copies of 12 tests is a pretty big pain.
https://github.com/llvm/llvm-project/pull/89620
    
    
More information about the llvm-commits
mailing list