[PATCH] D119752: [FPEnv][FMF] Move helper function to header, move fast math flags to new include file.

Kevin P. Neal via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 14 11:37:15 PST 2022


kpn created this revision.
kpn added reviewers: spatel, sepavloff.
Herald added subscribers: dexonsmith, hiraditya.
kpn requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

In a prior review I was asked to move the helper function canIgnoreSNaN() out to FPEnv.h. This wasn't possible at the time because that function needs the fast math flags, and including them includes lots of other stuff that isn't needed.

This patch moves the fast math flags out into a new FMF.h file unchanged, and moves the helper function out to FPEnv.h also unchanged. This ticket only moves code around.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D119752

Files:
  llvm/include/llvm/IR/FMF.h
  llvm/include/llvm/IR/FPEnv.h
  llvm/include/llvm/IR/Operator.h
  llvm/lib/Analysis/InstructionSimplify.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D119752.408528.patch
Type: text/x-patch
Size: 9322 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220214/25522d9d/attachment.bin>


More information about the llvm-commits mailing list