Don't pass ForDefinition_t in places it is redundant.
authorRafael Espindola <rafael.espindola@gmail.com>
Wed, 7 Feb 2018 19:04:41 +0000 (19:04 +0000)
committerRafael Espindola <rafael.espindola@gmail.com>
Wed, 7 Feb 2018 19:04:41 +0000 (19:04 +0000)
commitdeb10becf39672dd72d3a4c0143db2838ec1ccb2
treebe1bc0f17bafb463b82b0113c95cf8280b8e53a4
parent123c749ee9973c20bbcb9ecfe6f9b0860b55bc51
Don't pass ForDefinition_t in places it is redundant.

I found this while looking at the ppc failures caused by the dso_local
change.

The issue was that the patch would produce the wrong answer for
available_externally. Having ForDefinition_t available in places where
the code can just check the linkage is a bit of a foot gun.

This patch removes the ForDefiniton_t argument in places where the
linkage is already know.

llvm-svn: 324499
clang/lib/CodeGen/CGDecl.cpp
clang/lib/CodeGen/CGVTT.cpp
clang/lib/CodeGen/CGVTables.cpp
clang/lib/CodeGen/CodeGenModule.cpp
clang/lib/CodeGen/CodeGenModule.h
clang/lib/CodeGen/ItaniumCXXABI.cpp
clang/lib/CodeGen/TargetInfo.cpp
clang/lib/CodeGen/TargetInfo.h