[PATCH] D33870: Expose IRMover flag

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 6 12:22:37 PDT 2017


pcc requested changes to this revision.
pcc added a comment.
This revision now requires changes to proceed.

I'm not yet convinced that we should take this. If changing the flag from true to false broke your frontend, it may mean that there is a bug in the IRMover that was exposed by your frontend. So we may be introducing a flag that means "do you want this set of bugs or that other set of bugs", which doesn't seem like a good idea to me.

Can you go into more detail as to how changing the flag broke your frontend?


Repository:
  rL LLVM

https://reviews.llvm.org/D33870





More information about the llvm-commits mailing list