vitalybuka accepted this revision.
vitalybuka added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang/lib/Driver/SanitizerArgs.cpp:384
+                                KernelAddress | Efficiency),
+      std::make_pair(SafeStack, Address | HWAddress | Leak | Thread | Memory |
+                                    KernelAddress | Efficiency)};
----------------
I guess we check only one direction.
E.g. we have std::make_pair(Address, Thread but no std::make_pair(Thread, 
Address

So either this line or changes above are not needed.



Repository:
  rC Clang

https://reviews.llvm.org/D43606



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to