[PATCH] D60602: [InferAddressSpaces] Add AS parameter to the pass factory

Kévin Petit via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 18 06:50:40 PDT 2019


kpet added a comment.

> But would it be possible to extract an .ll file generated from C++ (may be with some amendments) and run opt with the pass enabled?

I did consider this as well. My understanding is that it would require further changes to the pass to add a command-line parameter that could be a third source for the flat address space. It would be a good thing to have but it wouldn't test my change to the pass factory.

> Are you expecting a certain IR pattern to occur with your change?

Not as such. This change is just plumbing.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60602





More information about the llvm-commits mailing list