ChuanqiXu added a comment.

(I don't why I can't send emails in the original mail address. It told me that 
your email addressed is not recorded by MX, while I don't know what is MX. So I 
tried to reply your email here)

This FIXME was added 6 years ago for Modules TS. 
(https://github.com/llvm/llvm-project/commit/becb92dec85924969ac0c3b049e0a74def431453).
And I moved the test simply without modifying it actually.

And I guess it may refer to the problem we are discussing. The spec says the 
implementation unit will import the primary module interface implicitly.
But the current implementation doesn't **import** it actually but loading it. 
This is inconsistent with the wording and it is not consistent with the design 
intention in clang.
It resulted some problems we saw.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126959

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

Reply via email to