Re: LLVM: use OpenBSD target code for AArch64

[Patrick Wildt]

On Fri, Nov 25, 2016 at 12:37:10AM +0100, Patrick Wildt wrote:
> Hi,
> 
> a clang binary using the AArch64 backend currently does not use
> proper OpenBSD defines and specifics.  This is because we don't
> tell clang to use OpenBSD target info if it encounters an AArch64
> target.
> 
> This diff makes LLVM do all the OpenBSD stuff for AArch64.
> 
> ok?
> 
> Patrick
> 
> diff --git a/gnu/llvm/tools/clang/lib/Basic/Targets.cpp 
> b/gnu/llvm/tools/clang/lib/Basic/Targets.cpp
> index 1b88184..d65c88a 100644
> --- a/gnu/llvm/tools/clang/lib/Basic/Targets.cpp
> +++ b/gnu/llvm/tools/clang/lib/Basic/Targets.cpp
> @@ -7551,6 +7551,8 @@ static TargetInfo *AllocateTarget(const llvm::Triple 
> ) {
>return new LinuxTargetInfo(Triple);
>  case llvm::Triple::NetBSD:
>return new NetBSDTargetInfo(Triple);
> +case llvm::Triple::OpenBSD:
> +  return new OpenBSDTargetInfo(Triple);
>  default:
>return new AArch64leTargetInfo(Triple);
>  }
> 

bump

Re: LLVM: use OpenBSD target code for AArch64

[Patrick Wildt]

On Fri, Nov 25, 2016 at 12:44:44AM +0100, Joerg Sonnenberger wrote:
> On Fri, Nov 25, 2016 at 12:37:10AM +0100, Patrick Wildt wrote:
> > This diff makes LLVM do all the OpenBSD stuff for AArch64.
> > 
> > ok?
> 
> If you plan to submit this upstream, you should also update the test
> cases accordingly.
> 
> Joerg
> 

As far as I know we did not import the tests, so there's no way I can
update those in our tree.

Patrick

Re: LLVM: use OpenBSD target code for AArch64

[Joerg Sonnenberger]

On Fri, Nov 25, 2016 at 12:37:10AM +0100, Patrick Wildt wrote:
> This diff makes LLVM do all the OpenBSD stuff for AArch64.
> 
> ok?

If you plan to submit this upstream, you should also update the test
cases accordingly.

Joerg

LLVM: use OpenBSD target code for AArch64

[Patrick Wildt]

Hi,

a clang binary using the AArch64 backend currently does not use
proper OpenBSD defines and specifics.  This is because we don't
tell clang to use OpenBSD target info if it encounters an AArch64
target.

This diff makes LLVM do all the OpenBSD stuff for AArch64.

ok?

Patrick

diff --git a/gnu/llvm/tools/clang/lib/Basic/Targets.cpp 
b/gnu/llvm/tools/clang/lib/Basic/Targets.cpp
index 1b88184..d65c88a 100644
--- a/gnu/llvm/tools/clang/lib/Basic/Targets.cpp
+++ b/gnu/llvm/tools/clang/lib/Basic/Targets.cpp
@@ -7551,6 +7551,8 @@ static TargetInfo *AllocateTarget(const llvm::Triple 
) {
   return new LinuxTargetInfo(Triple);
 case llvm::Triple::NetBSD:
   return new NetBSDTargetInfo(Triple);
+case llvm::Triple::OpenBSD:
+  return new OpenBSDTargetInfo(Triple);
 default:
   return new AArch64leTargetInfo(Triple);
 }