[PATCH] D46452: [sanitizer] Don't add --export-dynamic for Myriad

2018-05-17 Thread Walter Lee via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC332635: [sanitizer] Dont add --export-dynamic for Myriad (authored by waltl, committed by ). Changed prior to commit: https://reviews.llvm.org/D46452?vs=147233=147355#toc Repository: rC Clang

[PATCH] D46452: [sanitizer] Don't add --export-dynamic for Myriad

2018-05-16 Thread Walter Lee via Phabricator via cfe-commits
waltl updated this revision to Diff 147233. waltl added a comment. Address CR comments Repository: rL LLVM https://reviews.llvm.org/D46452 Files: clang/lib/Driver/ToolChains/CommonArgs.cpp Index: clang/lib/Driver/ToolChains/CommonArgs.cpp

[PATCH] D46452: [sanitizer] Don't add --export-dynamic for Myriad

2018-05-15 Thread Aleksey Shlyapnikov via Phabricator via cfe-commits
alekseyshl added inline comments. Comment at: clang/lib/Driver/ToolChains/CommonArgs.cpp:706 // to be dynamic to be sure we export sanitizer interface functions. - if (AddExportDynamic) + if (AddExportDynamic && TC.getTriple().getVendor() != llvm::Triple::Myriad)

[PATCH] D46452: [sanitizer] Don't add --export-dynamic for Myriad

2018-05-04 Thread Walter Lee via Phabricator via cfe-commits
waltl created this revision. waltl added reviewers: vitalybuka, eugenis, alekseyshl. Herald added a subscriber: llvm-commits. This is to work around a bug in some versions of gnu ld, where --export-dynamic implies -shared even if -static is explicitly given. Myriad supports static linking only,