jrtc27 added a comment.

In D149867#4387189 <https://reviews.llvm.org/D149867#4387189>, @glaubitz wrote:

> In D149867#4386271 <https://reviews.llvm.org/D149867#4386271>, @jrtc27 wrote:
>
>> I disagree. Being experimental doesn't mean you should do the wrong thing. 
>> Reusing stdcall in the frontend is ugly, pollutes non-m68k code paths (doing 
>> your own thing _avoids_ that and makes the experimental backend get out of 
>> the way, even) and introduces a bug where you can write garbage code and 
>> have it compile rather than be rejected like it should be.
>
> Maybe we can get the TLS stuff merged first before dealing with this more 
> complex change?

They're totally independent, so nothing's blocking that being merged. Plus this 
isn't a complex change to make, just a bunch of copy pasta.


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

https://reviews.llvm.org/D149867

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D149867:... Min-Yih Hsu via Phabricator via cfe-commits
    • [PATCH] D14... Sheng via Phabricator via cfe-commits
    • [PATCH] D14... Jessica Clarke via Phabricator via cfe-commits
    • [PATCH] D14... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D14... Jessica Clarke via Phabricator via cfe-commits

Reply via email to