Skip to content

Conversation

@dan-zheng
Copy link
Contributor

  • Create SILSerializer::DifferentiabilityWitnessesToEmit to track
    differentiability witnesses referenced by
    differentiability_witness_function instructions.
    These witnesses need to be serialized.
  • Move differentiability witness serialization before SIL function serialization
    but after visiting SIL functions (differentiability_witness_function
    instructions).
  • Use -emit-sorted-sil in test/AutoDiff/sil_differentiability_witness.sil
    for deterministic ordering for printing and deserialization.

- Create `SILSerializer::DifferentiabilityWitnessesToEmit` to track
  differentiability witnesses referenced by
  `differentiability_witness_function` instructions.
  These witnesses need to be serialized.
- Move differentiability witness serialization before SIL function serialization
  but after visiting SIL functions (`differentiability_witness_function`
  instructions).
- Use `-emit-sorted-sil` in test/AutoDiff/sil_differentiability_witness.sil
  for deterministic ordering for printing and deserialization.
@dan-zheng dan-zheng added the tensorflow This is for "tensorflow" branch PRs. label Nov 24, 2019
@dan-zheng dan-zheng requested review from marcrasi and rxwei November 24, 2019 03:33
@dan-zheng
Copy link
Contributor Author

@swift-ci Please clean test tensorflow

Copy link
Contributor

@rxwei rxwei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@dan-zheng
Copy link
Contributor Author

Closing. Mirror patch #28463 was merged to tensorflow.

@dan-zheng dan-zheng closed this Nov 24, 2019
@dan-zheng dan-zheng deleted the cocoa-puffs branch November 24, 2019 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tensorflow This is for "tensorflow" branch PRs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants