Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/23943 )

Change subject: [codegen] fix build on macOS
......................................................................

[codegen] fix build on macOS

This addresses linker error on macOS:

    void google::MakeCheckOpValueString<std::nullptr_t>(...), referenced from:
        ... google::MakeCheckOpString<std::nullptr_t, void*>(...) in 
jit_frame_manager.cc.o
  ld: symbol(s) not found for architecture x86_64

This is a follow-up to 0efbebd487e7d0c7f21ab0d5a91e0fa70046af83.

Change-Id: I00743d0a452ab5e33c3441cac4e06ff1bf58b7a6
Reviewed-on: http://gerrit.cloudera.org:8080/23943
Tested-by: Alexey Serbin <[email protected]>
Reviewed-by: Abhishek Chennaka <[email protected]>
Reviewed-by: Marton Greber <[email protected]>
Reviewed-by: Zoltan Martonka <[email protected]>
Reviewed-by: Zoltan Chovan <[email protected]>
---
M src/kudu/codegen/jit_frame_manager.cc
1 file changed, 3 insertions(+), 4 deletions(-)

Approvals:
  Alexey Serbin: Verified
  Abhishek Chennaka: Looks good to me, but someone else must approve
  Marton Greber: Looks good to me, but someone else must approve
  Zoltan Martonka: Looks good to me, but someone else must approve
  Zoltan Chovan: Looks good to me, approved

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I00743d0a452ab5e33c3441cac4e06ff1bf58b7a6
Gerrit-Change-Number: 23943
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Reviewer: Zoltan Martonka <[email protected]>

Reply via email to