[PATCH] D55028: [gn build] Add template for running llvm-tblgen and use it to add build file for llvm/lib/IR.
    Petr Hosek via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Nov 29 14:06:00 PST 2018
    
    
  
phosek accepted this revision.
phosek added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: llvm/utils/gn/secondary/llvm/utils/TableGen/tablegen.gni:37
+  action(target_name) {
+    forward_variables_from(invoker, [ "visibility" ])
+
----------------
thakis wrote:
> phosek wrote:
> > You should probably forward `testonly` as well`.
> I haven't needed it yet -- are you aware of any test-only tblgen invocations?
Nope, but it's a general pattern to always forward `visibility` and `testonly`.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55028/new/
https://reviews.llvm.org/D55028
    
    
More information about the llvm-commits
mailing list