[PATCH] [PECOFF][ELF][Darwin] Changes to fix some real issues in code

Shankar Kalpathi Easwaran shankarke at gmail.com
Mon Oct 28 07:22:29 PDT 2013


  <---------------snip------------>

  Thank you for sending the patches, but I'm still having trouble applying them. What I did was syncing to r193302 and then run git am. Patch 0004 failed. I also synced to HEAD and run git am. The patch were cleanly applied, but it couldn't compile. If you are familiar with git workflow and have an externally-accessible git repository, you might want to create a branch for it and push it to the external repository, so that I can pull and test it locally. That might be convenient. Patches are of course OK, though.
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  <---------------/snip-------------->

  Can you please refer to this mail snip. I was under the impression that the patch is ok from the mail as well as there were no test failures. If you think that the changes are not fine, please do go ahead and revert the changes (or) could you let me know about the changes so that I can understand the issue and fix it in the later patches.

  As for the tests, the tests do pass, am confused now. Are there any silent failures that you are seeing when the patches are applied.

  As for the main RoundTripPatch, let me know I could delete the lines from the driver that adds the RoundTrip Passes. This would make all of the current tests pass too. It might be easier to fix the git issues that you are seeing with the patches that I have kept sending to you. Let me know if you agree ?

http://llvm-reviews.chandlerc.com/D2022



More information about the llvm-commits mailing list