<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN">

<HTML>

<HEAD>

  <META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8">

  <META NAME="GENERATOR" CONTENT="GtkHTML/3.0.9">

</HEAD>

<BODY>

On Fri, 2003-11-14 at 10:03, Alkis Evlogimenos wrote:

<BLOCKQUOTE TYPE=CITE>

<PRE><FONT COLOR="#301fe4"><I>Just a comment on llvm headers. We currently use:

#include "llvm/Codegen/LiveVariables.h"

which causes an extra unnecessary lookup to compared to:

#include <llvm/Codegen/LiveVariables.h>

because it looks for the header file in the directory of the source file it 

includes it first before looking at the rest of the include path. Of course 

the header will never be there since the full path is specified. 

#include "" should only be used when headers are specified using relative 

paths. In our case the majority of header inclusions (if not all) use 

relative paths so we may want to consider either converting all our #include 

"" to #include <> or change header file inclusions to use relative paths. I 

don't see any advantages of one over the other but what we have today is not 

strictly correct :-)</I></FONT></PRE>

</BLOCKQUOTE>

<BR>

I couldn't agree more :)

</BODY>

</HTML>