This is an archive of the discontinued LLVM Phabricator instance.

Differential D17669

[zorg] Fixing an incorrect variable name.
ClosedPublic

Authored by sqlbyme on Feb 26 2016, 5:53 PM.

Details

Reviewers
gkistanova
Summary

I missed renaming this in the last patch.

Diff Detail

Event Timeline

sqlbyme updated this revision to Diff 49271.Feb 26 2016, 5:53 PM
sqlbyme retitled this revision from to [zorg] Fixing an incorrect variable name..
sqlbyme updated this object.
sqlbyme added a reviewer: gkistanova.
sqlbyme added a project: Restricted Project.
Herald added a subscriber: mehdi_amini. · View Herald TranscriptFeb 26 2016, 5:53 PM
gkistanova accepted this revision.Feb 26 2016, 7:45 PM
gkistanova edited edge metadata.

LGTM

Thanks

Galina

This revision is now accepted and ready to land.Feb 26 2016, 7:45 PM
gkistanova added a comment.Feb 26 2016, 7:47 PM

Committed r262107.

sqlbyme closed this revision.Feb 26 2016, 7:50 PM

Revision Contents

PathSize
zorg/
buildbot/
builders/
2 lines

Diff 49271

zorg/buildbot/builders/ClangLTOBuilder3Stage.py

Context not available.
if exit_status: if exit_status:
return {} return {}
k,v = stdout.split('=') k,v = stdout.split('=')
llvm_ver = v.strip().strip("\'\"") clang_ver = v.strip().strip("\'\"")
return { 'clang_ver' : clang_ver } return { 'clang_ver' : clang_ver }
f.addStep( f.addStep(
Context not available.