[PATCH] D26415: [XRay] Support AArch64 in Clang
Serge Rogatch via cfe-commits
cfe-commits at lists.llvm.org
Wed Nov 9 14:02:26 PST 2016
rSerge added inline comments.
================
Comment at: lib/Driver/Tools.cpp:4903-4906
if (Triple.getOS() == llvm::Triple::Linux &&
(Triple.getArch() == llvm::Triple::arm ||
- Triple.getArch() == llvm::Triple::x86_64)) {
+ Triple.getArch() == llvm::Triple::x86_64 ||
+ Triple.getArch() == llvm::Triple::aarch64)) {
----------------
dberris wrote:
> I'm wondering whether it's worth turning this into a `switch` statement now that we have more than two supported architectures?
I think that would lead to more awkward code: there wouldn't be a single decision outcome point (like the current `else` block), so to avoid duplicating the code which currently prints the message, a `bool` variable would be needed. I think it's more neat to just enumerate all the OS&CPU combinations in the `if` condition for now.
This place is not performance-critical and the compiler should convert appropriate `if`s into `switch`es anyway.
https://reviews.llvm.org/D26415
More information about the cfe-commits
mailing list