[PATCH] D47380: Make getStrictFPOpcodeAction(...) more accessible
Ulrich Weigand via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 29 09:09:55 PDT 2018
uweigand added a comment.
Is there a reason for dropping the "FP" from "StrictFP" in the method name? I think it would be better to keep it (i.e. use "getStrictFPOperationAction"), in particular since there are already other public methods in this area that use "StrictFP", like isStrictFPOpcode and MutateStrictFPToFP.
Otherwise this LGTM.
Repository:
rL LLVM
https://reviews.llvm.org/D47380
More information about the llvm-commits
mailing list