[cfe-commits] r143685 - /cfe/trunk/lib/Driver/Tools.cpp

Chandler Carruth chandlerc at gmail.com
Fri Nov 4 00:12:58 PDT 2011

Author: chandlerc
Date: Fri Nov  4 02:12:58 2011
New Revision: 143685

URL: http://llvm.org/viewvc/llvm-project?rev=143685&view=rev
Add a FIXME to cleanup the CPATH handling. It's currently burried with
the rest of the mess in InitHeaderSearch.cpp. We could hoist it into the
driver profitably, removing more noise from the driver -> frontend


Modified: cfe/trunk/lib/Driver/Tools.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=143685&r1=143684&r2=143685&view=diff
--- cfe/trunk/lib/Driver/Tools.cpp (original)
+++ cfe/trunk/lib/Driver/Tools.cpp Fri Nov  4 02:12:58 2011
@@ -403,6 +403,8 @@
   Args.AddAllArgs(CmdArgs, options::OPT_fauto_module_import);
   // Parse additional include paths from environment variables.
+  // FIXME: We should probably sink the logic for handling these from the
+  // frontend into the driver. It will allow deleting 4 otherwise unused flags.
   // CPATH - included following the user specified includes (but prior to
   // builtin and standard includes).
   AddIncludeDirectoryList(Args, CmdArgs, "-I", ::getenv("CPATH"));

More information about the cfe-commits mailing list