Michael Ho has posted comments on this change. Change subject: Fix LLVM linker bug ......................................................................
Patch Set 2: (2 comments) I haven't looked at the linker part of LLVM so it's taking me some time to undestand what's going on. http://gerrit.cloudera.org:8080/#/c/2656/2//COMMIT_MSG Commit Message: Line 25: istinct nit: distinct http://gerrit.cloudera.org:8080/#/c/2656/2/source/llvm/llvm-3.8.0-patches/0001-llvm-link-struct-types.patch File source/llvm/llvm-3.8.0-patches/0001-llvm-link-struct-types.patch: Line 53: if (Ty->isOpaque()) Why the distinction between opaque vs non-opaque type at this function before this change ? Why is it okay to kind of get rid of this here ? -- 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: 2 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: Yes
