[PATCH] D133943: [Utils] Match function return type and other attributes on the definition in update_cc_test_checks
    Johannes Doerfert via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Oct 31 09:55:17 PDT 2022
    
    
  
jdoerfert added a comment.
In D133943#3823710 <https://reviews.llvm.org/D133943#3823710>, @arichardson wrote:
> I have also wanted this in the past, and I don't think the church should be too bad since many tests don't use this flag and if they do it will only affect a few lines without any knock on effects.
I am not 100% sure this is the case. Could we check? Grep all the tests for the flag, run the update script on them as a NFC commit after this one. Verify they all run properly and the churn is not too bad.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D133943/new/
https://reviews.llvm.org/D133943
    
    
More information about the llvm-commits
mailing list