[PATCH] D109359: [TableGen] Emit a warning for unused template args
    Geoffrey Martin-Noble via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Nov  4 10:17:50 PDT 2021
    
    
  
GMNGeoffrey added a comment.
In D109359#3109384 <https://reviews.llvm.org/D109359#3109384>, @Paul-C-Anagnostopoulos wrote:
> I suggested an 'unused' statement above.
Yeah something like that seems fine to me. Also fine with a useless assignment, if such a thing is possible
> With it, we should be able to make this option on by default.
It appears to already be on by default :-(
> I don't think it's a good idea to completely disallow unused arguments.
Agreed.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109359/new/
https://reviews.llvm.org/D109359
    
    
More information about the llvm-commits
mailing list