Added support for Apple Silicon (M1).

bloomberg / bde

Basic Development Environment - a set of foundational C++ libraries used at Bloomberg.

Apache License 2.0

1.68k stars 318 forks source link

Added support for Apple Silicon (M1). #273

Closed emeryberger closed 1 year ago

emeryberger commented 3 years ago

Describe your changes The existing compiler flags did not allow builds on Apple Silicon platforms (e.g., the M1 mini). This PR rectifies this situation, adding __arm64 detection and ensuring it is treated as a 64-bit platform.

Testing performed Run with a test that uses the BDE library.

osubboo commented 3 years ago

Noted. We'll look into it.

mversche commented 2 years ago

Should this PR be closed in preference to #275?

emeryberger commented 2 years ago

That does look like a more comprehensive PR than this one (I was just about to test).

emeryberger commented 2 years ago

I updated to a more recent BDE and it no longer builds on the M1. I can work on it but if the other PR already works for a more recent version of BDE, it'd make more sense to merge that one. Let me know.