ABataev added a comment. In D101849#2764798 <https://reviews.llvm.org/D101849#2764798>, @jdoerfert wrote:
> In D101849#2764703 <https://reviews.llvm.org/D101849#2764703>, @mikerice > wrote: > >>> // RUN: %clang_cc1 -verify -triple x86_64-apple-darwin10 -O1 -fopenmp-simd >>> -emit-llvm %s -o - | FileCheck %s --check-prefix=CHECK10 >> >> Is there a good reason to run this with -O1? Doing so makes it super >> sensitive to which llvm passes run and now this test fails for us now with >> x86 as well. > > One of the O1 <https://reviews.llvm.org/owners/package/1/> was introduced by > 6e8248fdad5fc59306beb286a3089fe401460826 and I cannot really tell why we > would need it. > @ABataev do you remember if the O1 > <https://reviews.llvm.org/owners/package/1/> was needed? > If not I'd suggest to remove the O1 > <https://reviews.llvm.org/owners/package/1/> run line or add > -disable-llvm-optzns to the run line. > If it is needed, we can have a simple parallel for test with manual check > lines. No, it is not required. Most probably, needed to simplify test checks, nothing else. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101849/new/ https://reviews.llvm.org/D101849 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
