[PATCH] D26415: [XRay] Support AArch64 in Clang

Dean Michael Berris via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 8 15:08:35 PST 2016


dberris 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)) {
----------------
I'm wondering whether it's worth turning this into a `switch` statement now that we have more than two supported architectures?


https://reviews.llvm.org/D26415





More information about the cfe-commits mailing list