The doc for reassociate has been outdated and inconsistent with its mechanism.
Its description about rank is wrong, which I update here.
Details
Details
Diff Detail
Diff Detail
Event Timeline
llvm/lib/Transforms/Scalar/Reassociate.cpp | ||
---|---|---|
21 | I think this comment shouldn't mention exact rank numbers. The important thing to convey at this level is that the rank ordering is constants, followed by arguments, followed by RPO order of instructions. |
I think this comment shouldn't mention exact rank numbers. The important thing to convey at this level is that the rank ordering is constants, followed by arguments, followed by RPO order of instructions.