[all-commits] [llvm/llvm-project] ea8678: Move floating point related entities to namespace ...

Serge Pavlov via All-commits all-commits at lists.llvm.org
Wed Nov 20 04:06:07 PST 2019


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: ea8678d1c78ecf6c719b4a9ff1aa8db0087401ca
      https://github.com/llvm/llvm-project/commit/ea8678d1c78ecf6c719b4a9ff1aa8db0087401ca
  Author: Serge Pavlov <sepavloff at gmail.com>
  Date:   2019-11-20 (Wed, 20 Nov 2019)

  Changed paths:
    A llvm/include/llvm/IR/FPEnv.h
    M llvm/include/llvm/IR/IRBuilder.h
    M llvm/include/llvm/IR/IntrinsicInst.h
    M llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
    M llvm/lib/IR/CMakeLists.txt
    A llvm/lib/IR/FPEnv.cpp
    M llvm/lib/IR/IntrinsicInst.cpp
    M llvm/unittests/IR/IRBuilderTest.cpp

  Log Message:
  -----------
  Move floating point related entities to namespace level

This is recommit of commit e6584b2b7b2d, which was reverted in
30e7ee3c4bac together with af57dbf12e54.
Original message is below.

Enumerations that describe rounding mode and exception behavior were
defined inside ConstrainedFPIntrinsic. It makes sense to use the same
definitions to represent the same properties in other cases, not only
in constrained intrinsics. It was however inconvenient as required to
include constrained intrinsics definitions even if they were not needed.
Also using long scope prefix reduced readability.

This change moves these definitioins to the namespace llvm::fp.
No functional changes.

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




More information about the All-commits mailing list