shafik marked an inline comment as done.
shafik added a subscriber: rsmith.
shafik added a comment.

@martong I have been experimenting w/ how we pass around `ImportDefinitionKind` 
and pushing it through to more places does not help. I also tried defaulting 
most locations to `IDK_Everything` to experiment to see if maybe I was missing 
some crucial point and no luck. So while it seemed like a good direction it has 
not worked out, unless I missed something in your idea.

So my current approach may be the best we have until we can do some larger 
refactoring.

I had explored trying to fix this from the codegen and sema side of things but 
after discussing this w/ @rsmith there is no simple fix there that would not 
require some large refactoring on the LLDB side.


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

https://reviews.llvm.org/D86660

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to