[PATCH] D71467: [FPEnv] Generate constrained FP comparisons from clang

John McCall via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 13 13:24:41 PST 2020


rjmccall added a comment.

Is this approach going to work with scope-local strictness?  We need a way to do a comparison that has the non-strict properties but appears in a function that enables strictness elsewhere.



================
Comment at: llvm/include/llvm/IR/IRBuilder.h:2342
         return Insert(Folder.CreateFCmp(P, LC, RC), Name);
     return Insert(setFPAttrs(new FCmpInst(P, LHS, RHS), FPMathTag, FMF), Name);
   }
----------------
Can you make a helper method for the common code in the non-constrained paths here?

Please document the difference between these two methods.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71467/new/

https://reviews.llvm.org/D71467





More information about the llvm-commits mailing list