[LLVMdev] Headers & Libraries

[Reid Spencer]

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

> 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 couldn't agree more :)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20031114/6132070e/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20031114/6132070e/attachment.sig>