[llvm-dev] [RFC] migrating past C++11
llvm-dev at lists.llvm.org
Thu Feb 7 11:01:52 PST 2019
It seems the CMake changes have landed; but the docs are still a bit out of date?
CMake.html talks about LLVM_FORCE_USE_OLD_TOOLCHAIN but not LLVM_TEMPORARILY_ALLOW_OLD_TOOLCHAIN.
I’m. Not sure how one updates the website’s docs, I had assumed the RST files would automatically get rebuilt and pushed? Agreed we want it fixed, but I don’t think it’s good reason to revert since the error message is pretty clear.
Huh. I also thought the website was kept up to date automagically, which is why I looked at the website not the source; but the source does have the new option. Never mind…
Also, it looks like LLVM_TEMPORARILY_ALLOW_OLD_TOOLCHAIN is not propagated down to the NATIVE configuration when you set LLVM_OPTIMIZED_TABLEGEN. If that's going to be a permanent deficiency, it should be mentioned in the docs as well.
Someone mentioned MSVC was having issues that way? https://reviews.llvm.org/rL353374#624722
That seems like general badness in the way that configuration is set up, no? It should probably get fixed separately
"Somebody else ought to get around to fixing that someday so why bother documenting something that might eventually change" does not seem like a very robust argument. I've answered a pile of questions from newcomers lately, I would prefer that people not add new reasons for the documentation to lie about how things work currently. Or more accurately, don't work, because you can't currently use both those options at once.
If someone does change how the optimized-tablegen feature works (or eliminates it entirely), surely they would be able to update the documentation accordingly?
So, please fix.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-dev