This is an archive of the discontinued LLVM Phabricator instance.

[Tests] Updated tests for D53342
AbandonedPublic

Authored by xbolva00 on Oct 24 2018, 12:50 PM.

Details

Reviewers
efriedma

Diff Detail

Event Timeline

xbolva00 created this revision.Oct 24 2018, 12:50 PM
efriedma added inline comments.Oct 24 2018, 1:02 PM
test/CodeGenOpenCL/amdgpu-nullptr.cl
513 ↗(On Diff #170956)

This shouldn't change. "null" pointers are potentially valid in non-zero address-spaces. (Sorry, I should have spotted this sooner.)

xbolva00 added inline comments.Oct 24 2018, 1:15 PM
test/CodeGenOpenCL/amdgpu-nullptr.cl
513 ↗(On Diff #170956)

Thanks, I will fix this case.

xbolva00 updated this revision to Diff 170963.Oct 24 2018, 1:17 PM
nhaehnle removed a subscriber: nhaehnle.Oct 27 2018, 8:07 AM
xbolva00 updated this revision to Diff 171906.Oct 31 2018, 6:41 AM
xbolva00 updated this revision to Diff 171908.Oct 31 2018, 6:44 AM
xbolva00 abandoned this revision.Aug 18 2019, 9:04 AM

Will be handled as NFC commit after D53342 lands.