This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Fix bug causing crash due to invalid opencl version metadata.
ClosedPublic

Authored by yaxunl on Jul 19 2016, 12:18 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

yaxunl updated this revision to Diff 64540.Jul 19 2016, 12:18 PM
yaxunl retitled this revision from to AMDGPU: Fix bug causing crash due to invalid opencl version metadata..
yaxunl updated this object.
yaxunl added reviewers: arsenm, nhaustov.
yaxunl added a subscriber: llvm-commits.
arsenm added inline comments.Jul 19 2016, 2:05 PM
lib/Target/AMDGPU/AMDGPUAsmPrinter.cpp
791 ↗(On Diff #64540)

Should use unsigned or uint16_t

yaxunl updated this revision to Diff 64566.Jul 19 2016, 2:34 PM
yaxunl edited edge metadata.
yaxunl marked an inline comment as done.

Revised by Matt's comments.

nhaustov accepted this revision.Jul 20 2016, 2:59 AM
nhaustov edited edge metadata.

Ok.

This revision is now accepted and ready to land.Jul 20 2016, 2:59 AM
yaxunl updated this revision to Diff 64687.Jul 20 2016, 7:27 AM
yaxunl edited edge metadata.

The last update missed a file. Update again.

This revision was automatically updated to reflect the committed changes.