[clang] cuda clang: Move nvptx-surface.cu test to CodeGenCUDA (PR #134758)
Artem Belevich via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 8 11:49:53 PDT 2025
================
@@ -2,6 +2,170 @@
// RUN: %clang_cc1 -triple nvptx64-unknown-unknown -fcuda-is-device -O3 -o - %s -emit-llvm | FileCheck %s
#include "Inputs/cuda.h"
+struct char1 {
----------------
Artem-B wrote:
Those are actually *useful* failures and expose real issues in those tests.
- correctly-rounded-div.cu defines their own float4. Should not do that for CUDA tests. Rename locally.
- offloading-entries.cu happens to define part of surface-related data structures that your change is defining. That should also be reconciled. It may just work if you remove local declarations and adjust checks appropriately.
- profile-coverage-mapping.cu - looks cosmetic. We've added more functions, so there are more coverage entries.
- propagate-attributes.cu Ouch. Including a C++ header under "extern C" is not the best idea. The test only cares about the names of the functions in the test, so it's just a matter of making sure that `extern "C"` applies only to them.
I think it's a good opportunity to clean things up. If I didn't miss the mis-placement of the test during the initial review, it would all pop up during the review. We're just doing it a bit later.
https://github.com/llvm/llvm-project/pull/134758
More information about the cfe-commits
mailing list