[PATCH] D113583: [clang] Fix armv7-quick build by hardcoding -triple=x86_64 in OOM test.

Adam Czachorowski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 10 10:21:00 PST 2021


adamcz created this revision.
adamcz added a reviewer: kadircet.
Herald added a subscriber: kristof.beyls.
adamcz requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

The test was added in 48bb5f4cbe8d5951c1153e469dc6713a122b7fa3 <https://reviews.llvm.org/rG48bb5f4cbe8d5951c1153e469dc6713a122b7fa3> and it
creates a very large array, which is too large for ARM. Making the array
smaller is not a good option, since we would need a very low ulimit and
could then hit it for other reasons.

Should fix the https://lab.llvm.org/buildbot/#/builders/171/builds/5851
failure.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D113583

Files:
  clang/test/SemaCXX/PR51712-large-array-constexpr-check-oom.cpp


Index: clang/test/SemaCXX/PR51712-large-array-constexpr-check-oom.cpp
===================================================================
--- clang/test/SemaCXX/PR51712-large-array-constexpr-check-oom.cpp
+++ clang/test/SemaCXX/PR51712-large-array-constexpr-check-oom.cpp
@@ -1,7 +1,7 @@
 // REQUIRES: shell
 // UNSUPPORTED: win32
 // RUN: ulimit -v 1048576
-// RUN: %clang_cc1 -std=c++11 -fsyntax-only -verify %s
+// RUN: %clang_cc1 -std=c++11 -fsyntax-only -verify -triple=x86_64 %s
 // expected-no-diagnostics
 
 // This used to require too much memory and crash with OOM.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D113583.386222.patch
Type: text/x-patch
Size: 576 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20211110/7cbe10ac/attachment.bin>


More information about the cfe-commits mailing list