[PATCH] D50913: [FPEnv] Don't need copysign/fabs/fneg constrained intrinsics
Andy Kaylor via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 25 08:39:47 PDT 2018
andrew.w.kaylor added a comment.
In https://reviews.llvm.org/D50913#1245120, @cameron.mcinally wrote:
> There hasn't been any strong objects, but I haven't seen many strong accepts either besides the few main stakeholders. I'm under the assumption that silence is a passive reject in situations like this.
>
> Should I keep pushing for it? It felt like I was beating a dead horse...
It's hard to interpret silence sometimes, but my experience has been that this community is rarely passive about rejection. I think there are good reasons to make another attempt at moving forward with the fneg instruction.
Repository:
rL LLVM
https://reviews.llvm.org/D50913
More information about the llvm-commits
mailing list