felipepiovezan wrote:

> @felipepiovezan I don't this patch as it is has any changes that are 
> testable. I haven't added any code that will emit a RuntimeLang, only added 
> the option to do that so I can add those callers downstream. In retrospect I 
> should've marked it as NFC.

I see. I think this is yet another argument for flipping the argument order: if 
the new thing were the last argument, it would have been even more evident this 
is NFC :) 

https://github.com/llvm/llvm-project/pull/72011
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to