[llvm-commits] [cfe-commits] [PATCH] Add llvm.fmuladd intrinsic.
lhames at gmail.com
Thu May 31 19:51:06 PDT 2012
Yep - I'm not sure what to do about that. At the moment all we have in the
way of defaults is the language options DefaultFPContract value. Perhaps
it's reasonable to have a per-target DefaultFPContract value that can
override the language default. When it comes to explicitly referencing the
pragma I'm not sure what the best way to handle that is - As I understand
it we won't know how many functions it applies to (and thus whether it's a
good candidate as a global default) until we've already parsed them.
On Thu, May 31, 2012 at 7:40 PM, John McCall <rjmccall at apple.com> wrote:
> On May 31, 2012, at 7:22 PM, Lang Hames wrote:
> > Thanks for the suggestion Matthieu. I spoke to Doug and he recommended
> using attributes rather than a FunctionDecl bit to represent the
> fp_contract state.
> Hmm. I had suggested a bit on FunctionDecl on the assumption that this
> would often be controlled globally, maybe by using a flag to control the
> default or by activating a #pragma before including all the headers.
> Actually, I could even imagine a target (maybe a GPU target?) even
> opting-in to this behavior by default. If we're going to use an Attr, we
> need to make sure it doesn't get added unless the current #pragma state is
> different from the global default; we really don't want to be allocating
> an attribute for every function definition in the translation unit.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits