[PATCH] D106065: [Clang] Add an empty builtins.h file.

Nemanja Ivanovic via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 15 10:11:11 PDT 2021


nemanjai added inline comments.


================
Comment at: clang/lib/Headers/builtins.h:8
+\*===----------------------------------------------------------------------===*/
+
+#ifndef __BUILTINS_H
----------------
Please describe the purpose of this header file in a comment here.


================
Comment at: clang/test/Headers/builtins-header.c:1
+// RUN: %clang_cc1 -triple powerpc64-unknown-unknown -ffreestanding -emit-llvm -o - %s | FileCheck %s
+// RUN: %clang_cc1 -triple powerpc64le-unknown-unknown -ffreestanding -emit-llvm -o - %s | FileCheck %s
----------------
Does this need something like `REQUIRES: powerpc-registered-target`?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D106065/new/

https://reviews.llvm.org/D106065



More information about the cfe-commits mailing list