nickdesaulniers accepted this revision. nickdesaulniers added a comment. This revision is now accepted and ready to land.
Looks reasonable. Can you give us some time to test this on the Linux kernel? ================ Comment at: clang/test/CodeGen/memcpy-inline-builtin.c:3 + +// RUN: %clang_cc1 -triple x86_64-unknown-unknown -S -emit-llvm -o - %s | FileCheck %s +// ---------------- are you able to leave off the `-unknown-unknown` from the triple? I assume those are the defaults; I know you can with `llc` but not sure about `clang`. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109967/new/ https://reviews.llvm.org/D109967 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
