[PATCH] [LoopVectorizer] if unsafe-fp-math FN attribute is set, enable reduction detection with float operations

Hal Finkel hfinkel at anl.gov
Fri Nov 28 09:43:03 PST 2014


Pekka,

What's your motivation for doing this? Aside from the fact that we can't yet set fast math flags on function calls (to sqrt, etc.), I think we're pretty close to removing the function attribute all together. The instruction-level flags play much nicer with LTO, and give more precision control to the frontends; it is not clear to me that overriding them with the function-level attribute is desirable.

 -Hal

----- Original Message -----
> From: "Pekka Jääskeläinen" <pekka.jaaskelainen at tut.fi>
> To: "Commit Messages and Patches for LLVM" <llvm-commits at cs.uiuc.edu>
> Sent: Friday, November 28, 2014 11:13:39 AM
> Subject: [PATCH] [LoopVectorizer] if unsafe-fp-math FN attribute is set, enable	reduction detection with float
> operations
> 
> Hi,
> 
> The attached patch checks the function attributes for
> "unsafe-fp-math" to relax FP constraints when detecting
> reduction operations.
> 
> --
> --Pekka
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> 

-- 
Hal Finkel
Assistant Computational Scientist
Leadership Computing Facility
Argonne National Laboratory




More information about the llvm-commits mailing list