[PATCH] D110776: A new hidden option exec-on-ir-change=exe that calls exe after each time IR changes

Jamie Schmeiser via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 2 11:08:47 PST 2022


jamieschmeiser updated this revision to Diff 479682.
jamieschmeiser added a comment.

@aeubanks When I first reverted this because it failed on windows, I wasn't sure what to do to fix it and it fell off my radar.
I have updated it to reflect changes since then (the tests were updated to reflect changes to specifying filtering, functions have moved, etc).  I've made a few minor changes, such as the functions for managing the temp files return Optional strings now and the tempfile name is different to reflect the multiple usage.  I've also added a lit.local.cfg file to ensure that '/bin/cat' exists, which should fix the original windows failure that caused the revert.  Can you please take a look to see that everything is still okay?  Thanks.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D110776/new/

https://reviews.llvm.org/D110776

Files:
  llvm/include/llvm/IR/PrintPasses.h
  llvm/include/llvm/Passes/StandardInstrumentations.h
  llvm/lib/IR/PrintPasses.cpp
  llvm/lib/Passes/StandardInstrumentations.cpp
  llvm/test/Other/ChangeTesters/exec-on-ir-change.ll
  llvm/test/Other/ChangeTesters/lit.local.cfg

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D110776.479682.patch
Type: text/x-patch
Size: 15823 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221202/b97048c3/attachment.bin>


More information about the llvm-commits mailing list