[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