[PATCH] D16514: Add -stop-on-failure driver option, and enable it by default for CUDA compiles.

Eric Christopher via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 28 12:13:57 PST 2016


echristo added a comment.

In general it feels like keeping 2 errors might make the most sense:

(Using a multiarch build rather than a cuda command line, but it should still be the same behavior for consistency)

t.c:

#if _NOT_ARCH4_
#error "aiee!"
#endif

clang -arch arch1 -arch arch2 -arch arch3 -arch arch4 t.c

seems like it might be nice to get 3 errors here rather than a single one and fixing that single one, then getting another one, etc. or realizing what the error is here.

I don't feel strongly about this, but I'm still uncertain as to why we want to make things more complicated here :)

-eric


http://reviews.llvm.org/D16514





More information about the cfe-commits mailing list