Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix tensor_or_memref build error without math.h #59775

Closed

Conversation

i-chaochen
Copy link
Contributor

#59536 (comment)

Since pevious PR has been rolled back, this one adopted reviewer's feedback to fix Mac build error.

Thanks @reedwm for the feedback and suggestion!

@google-ml-butler google-ml-butler bot added the size:XS CL Change Size: Extra Small label Feb 22, 2023
@google-ml-butler google-ml-butler bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Feb 22, 2023
@reedwm
Copy link
Member

reedwm commented Feb 22, 2023

Thanks for the fix!

@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Feb 22, 2023
copybara-service bot pushed a commit to tensorflow/mlir-hlo that referenced this pull request Feb 23, 2023
Imported from GitHub PR tensorflow/tensorflow#59775

tensorflow/tensorflow#59536 (comment)

Since pevious PR has been rolled back, this one adopted reviewer's feedback to fix Mac build error.

Thanks @reedwm for the feedback and suggestion!
Copybara import of the project:

--
4f9f856e0159f5dd7c4e8d0e3b5232e55795f700 by Chao Chen <cchen104@amd.com>:

fix tensor_or_memref build error without math.h

Merging this change closes #59775

PiperOrigin-RevId: 511716904
copybara-service bot pushed a commit to openxla/xla that referenced this pull request Feb 23, 2023
Imported from GitHub PR tensorflow/tensorflow#59775

tensorflow/tensorflow#59536 (comment)

Since pevious PR has been rolled back, this one adopted reviewer's feedback to fix Mac build error.

Thanks @reedwm for the feedback and suggestion!
Copybara import of the project:

--
4f9f856e0159f5dd7c4e8d0e3b5232e55795f700 by Chao Chen <cchen104@amd.com>:

fix tensor_or_memref build error without math.h

Merging this change closes #59775

PiperOrigin-RevId: 511716904
@gbaned gbaned added the comp:xla XLA label Feb 23, 2023
@gbaned gbaned added this to Assigned Reviewer in PR Queue via automation Feb 23, 2023
PR Queue automation moved this from Assigned Reviewer to Closed/Rejected Feb 23, 2023
@google-ml-butler google-ml-butler bot removed the ready to pull PR ready for merge process label Feb 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:xla XLA size:XS CL Change Size: Extra Small
Projects
PR Queue
  
Closed/Rejected
Development

Successfully merging this pull request may close these issues.

None yet

4 participants