-
Notifications
You must be signed in to change notification settings - Fork 229
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
arch: Support more platforms #2415
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2415 +/- ##
==========================================
- Coverage 86.74% 86.72% -0.03%
==========================================
Files 235 235
Lines 44701 44724 +23
Branches 8288 8293 +5
==========================================
+ Hits 38777 38786 +9
- Misses 5195 5212 +17
+ Partials 729 726 -3 ☔ View full report in Codecov by Sentry. |
devito/arch/compiler.py
Outdated
return super().__getitem__(key) | ||
|
||
def has_key(self, k): | ||
return k in self.keys() or k.startswith('gcc-') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no need for "()" after keys
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes you do
I've also changed the PR title to start with "arch: ..." rather than "compiler: ..." |
Reinstating Graviton runner so this can actually be tested. |
a5e9828
to
f9b08f0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GTG now
Generalizes gcc
suport graviton 2-3-4 and add cflags to use gcc with it as well