[PATCH] D77247: Merge FloatingPointMode and FPEnv headers

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 2 05:56:12 PDT 2020


arsenm added a comment.

In D77247#1955942 <https://reviews.llvm.org/D77247#1955942>, @cameron.mcinally wrote:

> The `fp::` qualifier is kind of verbose. I'm not familiar with LLVM's C++ rules, but can we replace them with `using namespace fp;`? Or can we remove the `fp` namespace completely?


I’m not sure, I just merged into FPEnv and followed what it was doing.


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

https://reviews.llvm.org/D77247





More information about the llvm-commits mailing list