[PATCH] D43606: [Driver] Add SafeStack to a map of incompatible sanitizers

[Vitaly Buka via Phabricator via cfe-commits]

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