[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