[llvm-commits] r55638 - /llvm/trunk/include/llvm/Function.h

[Devang Patel]

On Sep 26, 2008, at 2:20 AM, Duncan Sands wrote:

> I think in general all these flags should be tri-state:
> "do it", "don't do it" or don't care.  So here you would
> have three possibilities: "x86.sse3", "x86.no-sse3" or
> nothing.  The point is that it may be useful to override
> the function note at codegen time.  For example if you
> specify sse3 to llc it can codegen all the "don't care"
> with sse3.


Yes, I agree.

During LTO, the linker may accept certain flags on the command line  
and can override notes. Nick touched this subject briefly earlier.

This is a policy decision enforced by  by llc or ld or other tools. In  
any case, when they override existing notes/attributes they should add/ 
remove/update notes/attributes in the IR first, not just silently tell  
code generator to ignore these notes/attributes.

-
Devang