[all-commits] [llvm/llvm-project] 8290f2: [FPEnv][FMF] Move helper function to header, move ...

Kevin P. Neal via All-commits all-commits at lists.llvm.org
Wed Feb 16 09:35:23 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8290f2535ba03120160881ca47e8466cbf7a8b26
      https://github.com/llvm/llvm-project/commit/8290f2535ba03120160881ca47e8466cbf7a8b26
  Author: Kevin P. Neal <kevin.neal at sas.com>
  Date:   2022-02-16 (Wed, 16 Feb 2022)

  Changed paths:
    A llvm/include/llvm/IR/FMF.h
    M llvm/include/llvm/IR/FPEnv.h
    M llvm/include/llvm/IR/Operator.h
    M llvm/lib/Analysis/InstructionSimplify.cpp

  Log Message:
  -----------
  [FPEnv][FMF] Move helper function to header, move fast math flags to new include file.

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.

Differential Revision: https://reviews.llvm.org/D119752




More information about the All-commits mailing list