[llvm] [IR] Remove the possibility of ConstantExpr having fast-math flags. (PR #94507)
Joshua Cranmer via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 5 11:57:11 PDT 2024
jcranmer-intel wrote:
> It would be good to also make the class inherit from Instruction instead of Operator (in this context, "Operator" means "Instruction or ConstantExpr").
I've considered that as well. `FPMathOperator` was originally an `Operator` when it was first implemented and `Instruction`-only, so I've reverted it back to that form for now. I've been poking around having FMF be present on `Instruction` in lieu of `SubclassOptionalData`, at which case the inheritance from `Instruction` is far more helpful. If that is the route taken to add more FMF bits, then I'll do it as part of that change.
https://github.com/llvm/llvm-project/pull/94507
More information about the llvm-commits
mailing list