Tim Armstrong has posted comments on this change.

Change subject: Fix LLVM linker bug
......................................................................


Patch Set 4: Code-Review+1

Updated since I ran the patch through clang-format as requested by upstream 
reviewer.

I'm still in the process of trying to convince the upstream reviewer that it is 
a bug and not expected behaviour. Michael, are you ok with giving a +2 and 
moving ahead with this?

-- 
To view, visit http://gerrit.cloudera.org:8080/2656
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I8aef7f79c8d905b267f5acc3392384b1b333e7d3
Gerrit-PatchSet: 4
Gerrit-Project: Toolchain
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Casey Ching <[email protected]>
Gerrit-Reviewer: Michael Ho
Gerrit-Reviewer: Michael Ho <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-HasComments: No

Reply via email to