[PATCH] D65994: Extended FPOptions with new attributes
Serge Pavlov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 12 20:48:05 PDT 2019
sepavloff marked 2 inline comments as done.
sepavloff added a comment.
In D65994#1625457 <https://reviews.llvm.org/D65994#1625457>, @rjmccall wrote:
> Since this setting changes IR output, you should write a test that emits IR for source code and tests that you get the right IR output.
This patch is a part of patch chain, it extends FPOptions with new options. In D65997 <https://reviews.llvm.org/D65997> `pragma clang fp` is extended to manipulate these options. Finally D66092 <https://reviews.llvm.org/D66092> modifies code generator so that it emit IR depending on the new options in FPOptions, it makes possible to write IR tests.
In D65994#1623088 <https://reviews.llvm.org/D65994#1623088>, @kpn wrote:
> In D65994#1622840 <https://reviews.llvm.org/D65994#1622840>, @aaron.ballman wrote:
> > In general, this seems reasonable, but is missing test code.
> The IRBuilder does have a strict FP mode setting now. When strict mode is enabled the (implemented) constrained intrinsics automatically replace the normal FP instructions. I wonder if that would be right for testing of this patch?
It is just what D66092 <https://reviews.llvm.org/D66092> does.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits