[PATCH] D14965: [llc/opt] Add an option to run all passes twice
Keno Fischer via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 1 05:22:41 PST 2015
loladiro updated this revision to Diff 41495.
loladiro added a comment.
Ok, how about doing this for the run-twice option:
- Clone the module to make sure the operation is idempotent
- Emit both results into buffers and always compare them to make sure they're equal
- Always emit the result of the second run
That way, to investigate a failure here, run it once without the run-twice option,
once with the run twice option and compare the outputs.
I also added some trivial tests, just to make sure these are exercised now that
they actually have non-trivial behavior.
http://reviews.llvm.org/D14965
Files:
test/MC/ELF/empty-twice.ll
test/Other/opt-twice.ll
tools/llc/llc.cpp
tools/opt/opt.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D14965.41495.patch
Type: text/x-patch
Size: 7314 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151201/c29797cf/attachment.bin>
More information about the llvm-commits
mailing list