[PATCH] propagate IR-level fast-math-flags to DAG nodes
Sanjay Patel
spatel at rotateright.com
Tue May 12 10:00:22 PDT 2015
For the record, r236546 was reverted by:
http://reviews.llvm.org/rL236600
The bug is independent of FMF or the other flags. It can be reproduced by allocating a plain SDNode for any instruction with 2 operands rather than a BinarySDNode. The bug manifests (visible under valgrind) after recycling the operand memory and/or morphing a node.
I didn't take any perf measurements, but I assume that we want to continue using the specialized operand classes (UnarySDNode, BinarySDNode, TernarySDNode) for their allocation optimizations. So I simplified this patch to keep the existing BinaryWithFlagsSDNode class that derives from BinarySDNode:
http://reviews.llvm.org/rL237046
REPOSITORY
rL LLVM
http://reviews.llvm.org/D8900
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list