[PATCH] D38740: [llvm-rc] Handle the /I option

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 10 18:23:31 PDT 2017


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

lgtm



================
Comment at: llvm/tools/llvm-rc/ResourceFileWriter.cpp:1439
+
+  if (auto Result =
+          llvm::sys::Process::FindInEnvPath("INCLUDE", File, Params.NoInclude))
----------------
Maybe add a similar comment?
  // 4. Last, search INCLUDE paths from the environment.


https://reviews.llvm.org/D38740





More information about the llvm-commits mailing list