[PATCH] D117778: [OpaquePtrs] Add -normalize-opaque-pointers option
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 20 13:34:56 PST 2022
nikic added a comment.
In D117778#3259584 <https://reviews.llvm.org/D117778#3259584>, @dblaikie wrote:
> Could this be done by roundtripping through llvm-dis, rather than by having a builtin feature? (even if it can be done that way, not necessarily suggesting it's the better option - adding extra program executions will slow down tests especially on windows, etc - but curious/considering all the options, etc)
This is actually what I originally proposed in https://reviews.llvm.org/D109290#2988806, and you suggested to integrate the option to save the extra program execution :) Another benefit of the option is that we can also drop the ptr to ptr bitcasts.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117778/new/
https://reviews.llvm.org/D117778
More information about the llvm-commits
mailing list