[PATCH] D109967: Simplify handling of builtin with inline redefinition

Nick Desaulniers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 21 13:53:20 PDT 2021


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



More information about the cfe-commits mailing list