[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