samsonov added inline comments.
================
Comment at: lib/CodeGen/CGExpr.cpp:2543
@@ +2542,3 @@
+ // RE: Bug: 25682
+ if(!getLangOpts().mergeTraps) {
+ llvm::InlineAsm *EmptyAsm =
llvm::InlineAsm::get(llvm::FunctionType::get(CGM.VoidTy, false),
----------------
danielaustin wrote:
> samsonov wrote:
> > Note that this will also affect `-ftrapv`, which might be unexpected, as we
> > mention "sanitize" in the flag name.
> Would it be better in your opinion to remove sanitize from the name, or check
> for the presence of the integer sanitizers here?
Probably former - we already have `-ftrap-function` which changes the behavior
of both sanitizer-generated and regular traps - this flag is no different.
Repository:
rL LLVM
http://reviews.llvm.org/D15208
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
