This is an archive of the discontinued LLVM Phabricator instance.

Terrible change as an example for CGO!
AbandonedPublic

Authored by chandlerc on Feb 7 2015, 9:49 AM.

Details

Reviewers
freik
Summary

This terrible ieda but demonstrates the code review system for LLVM

Diff Detail

Event Timeline

chandlerc updated this revision to Diff 19533.Feb 7 2015, 9:49 AM
chandlerc retitled this revision from to Terrible change as an example for CGO!.
chandlerc updated this object.
chandlerc edited the test plan for this revision. (Show Details)
chandlerc added a subscriber: Unknown Object (MLST).

This is not a code review, it is a bad example!!!

lib/Transforms/Scalar/SROA.cpp
163–166

aoeurschaoeursch

freik requested changes to this revision.Feb 7 2015, 9:56 AM
freik added a reviewer: freik.
freik added a subscriber: freik.

Thanks for doing the LLVM tutorial, but this code clearly needs more work :)

lib/Transforms/Scalar/SROA.cpp
166

Wonderful sample code.

This revision now requires changes to proceed.Feb 7 2015, 9:56 AM
chandlerc abandoned this revision.Apr 6 2016, 10:22 PM