[libc-commits] [PATCH] D90985: [libc] Extend WrapperGen to emit aliases.

Michael Jones via Phabricator via libc-commits libc-commits at lists.llvm.org
Fri Nov 6 15:59:13 PST 2020


michaelrj accepted this revision.
michaelrj added a comment.
This revision is now accepted and ready to land.

lgtm with nits



================
Comment at: libc/utils/tools/WrapperGen/Main.cpp:65
+    // If not emitting an alias, and not appending to another file,
+    // we should include the implementtion header to ensure the wrapper
+    // compiles.
----------------
nit: implementation


================
Comment at: libc/utils/tools/WrapperGen/Main.cpp:77-78
+    if (!ErrorOrBuf) {
+      llvm::PrintFatalError("Unable to read the file " + AppendToFile +
+                            " to append to.");
+    }
----------------
nit: add single quotes around the filename for clarity


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D90985/new/

https://reviews.llvm.org/D90985



More information about the libc-commits mailing list