[LLVMdev] r156323 - Reassociate FP operands.
Chad Rosier
mcrosier at codeaurora.org
Mon Jun 30 12:25:24 PDT 2014
Owen/All,
I've been working on adding support for reassociation with unsafe math
(see: http://reviews.llvm.org/D4129).
Do you know if this change, r156323, is still necessary? Specifically, do
we need the reassociation pass to canonicalize FP operands for CSE to work
effectively? This kinda scares me if it does! :(
Side note:
Without this canonicalization, I did see a 3% regression in Mesa because
the slp-vectorizer's cost model is sensitive to the ordering of operands.
I'm working on reducing a test case now.
Chad
More information about the llvm-dev
mailing list