arichardson added a comment.

Thanks documentation looks much better now.



================
Comment at: clang/docs/ClangCommandLineReference.rst:907
+
+.. option:: -fsemantic-interposition, -fno-semantic-interposition
+
----------------
arichardson wrote:
> arichardson wrote:
> > This looks like it should be reordered?
> Sorry I mean that it looks like all other entries are `.. option::` followed 
> by the description but here it's the other way around. It seems like this 
> change will make the documention of -fsanitize-* be `Enable semantic 
> interposition`.
It still looks like the documentation here is for the wrong option. I think it 
should be:

```
.. option:: -fsanitize=<check>,<arg2>..., -fno-sanitize=<arg1>,<arg2>...

Turn on runtime checks for various forms of undefined or suspicious behavior. 
See user manual for available checks

.. option:: -fno-semantic-interposition, -fsemantic-interposition

Enable semantic interposition. Semantic interposition allows for the
interposition of a symbol by another at run time, thus preventing a range of
inter-procedural optimisation.
```




Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72829/new/

https://reviews.llvm.org/D72829



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

Reply via email to